Skip to content
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(test-tooling): add new besu AIO image builder utility #3528

Merged

Conversation

petermetz
Copy link
Member

  1. This will help author test cases for the Besu connector that are designed
    to be testing the latest version of the image instead of a pinned one that
    we pull from the registry as part of the test case.
  2. We need tests for both latest and pinned image versions if we want to
    make sure that the connector is backward compatible with older ledger versions.

Signed-off-by: Peter Somogyvari peter.somogyvari@accenture.com

Pull Request Requirements

  • Rebased onto upstream/main branch and squashed into single commit to help maintainers review it more efficient and to avoid spaghetti git commit graphs that obfuscate which commit did exactly what change, when and, why.
  • Have git sign off at the end of commit message to avoid being marked red. You can add -s flag when using git commit command. You may refer to this link for more information.
  • Follow the Commit Linting specification. You may refer to this link for more information.

Character Limit

  • Pull Request Title and Commit Subject must not exceed 72 characters (including spaces and special characters).
  • Commit Message per line must not exceed 80 characters (including spaces and special characters).

A Must Read for Beginners
For rebasing and squashing, here's a must read guide for beginners.

petermetz added a commit to petermetz/cacti that referenced this pull request Sep 8, 2024
Depends on hyperledger#3528 => test(test-tooling): add new besu AIO image builder utility

Fixes hyperledger#3527

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
Copy link
Contributor

@jagpreetsinghsasan jagpreetsinghsasan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

1. This will help author test cases for the Besu connector that are designed
to be testing the latest version of the image instead of a pinned one that
we pull from the registry as part of the test case.
2. We need tests for both latest and pinned image versions if we want to
make sure that the connector is backward compatible with older ledger versions.

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
@petermetz petermetz force-pushed the test-tooling-besu-aio-image-builder branch from 1330b49 to 8c6b37c Compare September 10, 2024 05:24
@petermetz petermetz merged commit 72fda32 into hyperledger:main Sep 10, 2024
147 of 148 checks passed
@petermetz petermetz deleted the test-tooling-besu-aio-image-builder branch September 10, 2024 06:19
petermetz added a commit to petermetz/cacti that referenced this pull request Sep 10, 2024
Depends on hyperledger#3528 => test(test-tooling): add new besu AIO image builder utility

Fixes hyperledger#3527

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
petermetz added a commit that referenced this pull request Sep 10, 2024
Depends on #3528 => test(test-tooling): add new besu AIO image builder utility

Fixes #3527

Signed-off-by: Peter Somogyvari <peter.somogyvari@accenture.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants