-
Notifications
You must be signed in to change notification settings - Fork 286
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test(connector-fabric): fix module requires Go 1.17 #914
Labels
Milestone
Comments
petermetz
added
bug
Something isn't working
Fabric
dependencies
Pull requests that update a dependency file
labels
May 4, 2021
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 4, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 4, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 4, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 6, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
AzaharaC
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 6, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
AzaharaC
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 6, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 14, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 14, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 20, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 20, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 20, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 20, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 21, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 21, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 21, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 21, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 21, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
jordigiam
pushed a commit
to kikoncuo/cactus
that referenced
this issue
May 21, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 25, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 25, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 25, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 25, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 27, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 27, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 28, 2021
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
to petermetz/cacti
that referenced
this issue
May 28, 2021
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
that referenced
this issue
May 28, 2021
This was initially a candidate for resolving the issue of #914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
petermetz
added a commit
that referenced
this issue
May 28, 2021
Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes #914 Signed-off-by: Peter Somogyvari <[email protected]>
RafaelAPB
pushed a commit
to RafaelAPB/blockchain-integration-framework
that referenced
this issue
Mar 9, 2022
This was initially a candidate for resolving the issue of hyperledger-cacti#914 but later that idea turned out to not work. At the end of the day we always thrive to work with the latest and greatest of all of our dependencies CVEs permitting so the upgrade can stay in as a valid change. Signed-off-by: Peter Somogyvari <[email protected]>
RafaelAPB
pushed a commit
to RafaelAPB/blockchain-integration-framework
that referenced
this issue
Mar 9, 2022
…#914 Primary change: ------------------ Pinned the buggy dependency to yesterday's version (the bug was introduced in this morning's build). This prevents today's version from being used and fixes the problem. Secondary change: -------------------- Upgraded the container image that's being used for the test to the one that has the fabric images pre-cached. This leads to faster test execution and lower probability of developers getting hit by the dreaded DockerHub rate limiting issue. Fixes hyperledger-cacti#914 Signed-off-by: Peter Somogyvari <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Describe the bug
The hello world contract we deploy for the 1.4.x tests via go sources does not have it's dependencies pinned in the tests so the auto-upgrade finally broke the test:
To Reproduce
Run the tests.
Expected behavior
Tests pass.
Logs/Stack traces
Cloud provider or hardware configuration:
Dev machine
Operating system name, version, build:
Use this command:
printf "$(uname -srm)\n$(cat /etc/os-release)\n"
For example
Hyperledger Cactus release version or commit (git rev-parse --short HEAD):
main @ today
Hyperledger Cactus Plugins/Connectors Used
Fabric
Additional context
N/A
cc: @takeutak @sfuji822 @hartm @jonathan-m-hamilton @AzaharaC @jordigiam @kikoncuo @petermetz @arnab-roy
The text was updated successfully, but these errors were encountered: