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

ci: image parameterization via env variables #366

Merged
merged 2 commits into from
Sep 16, 2024

Conversation

rabi-siddique
Copy link
Contributor

@rabi-siddique rabi-siddique commented Sep 16, 2024

This PR introduces the ability to input a custom image name for E2E testing.

Previous Setup:

  • Previously, we used the image ghcr.io/agoric/agoric-3-proposals for E2E testing.
  • We could specify the tag for this image during testing in CI via an input.

The Issue:

  • Recently, images like ghcr.io/agoric/agoric-sdk:a3p-use-upgrade-next-20240911203722-bc66a5 started being used for E2E testing, in u17.
  • To test this image in CI, we manually hardcoded the image name in the docker-compose file (as done in testing with u17 #365). This was not ideal.

The Fix:

  • This PR solves the issue by allowing us to input the image name directly to test different images in CI.

@rabi-siddique rabi-siddique added liquidation-testing Trigger E2E tests for Liquidation liquidation-reconstitution-testing Trigger E2E tests for Liquidation-Reconstitution labels Sep 16, 2024
Copy link

cloudflare-workers-and-pages bot commented Sep 16, 2024

Deploying dapp-inter-test with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2984151
Status: ✅  Deploy successful!
Preview URL: https://81fe999a.dapp-inter-test.pages.dev
Branch Preview URL: https://rs-passing-a-custom-image-fo.dapp-inter-test.pages.dev

View logs

Copy link

cloudflare-workers-and-pages bot commented Sep 16, 2024

Deploying agoric-dapp-inter-emerynet with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2984151
Status: ✅  Deploy successful!
Preview URL: https://7d5ede7c.agoric-dapp-inter-emerynet.pages.dev
Branch Preview URL: https://rs-passing-a-custom-image-fo.agoric-dapp-inter-emerynet.pages.dev

View logs

Copy link

github-actions bot commented Sep 16, 2024

Network:
Commit: 429eeae
Ref: refs/heads/main
IPFS v1 hash: bafybeifw2fsj7v6jhbyfhb5cubt3d6md6vb6z6vwhh4mkkzgqet42eexle
CF - DWeb - 4EVERLAND

@rabi-siddique rabi-siddique force-pushed the rs-passing-a-custom-image-for-agd branch from 3756f91 to 2984151 Compare September 16, 2024 05:34
@rabi-siddique rabi-siddique merged commit 429eeae into main Sep 16, 2024
6 of 9 checks passed
@rabi-siddique rabi-siddique deleted the rs-passing-a-custom-image-for-agd branch September 16, 2024 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
liquidation-reconstitution-testing Trigger E2E tests for Liquidation-Reconstitution liquidation-testing Trigger E2E tests for Liquidation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants