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: ensure API is only updated if image and measurements are uploaded #2413

Merged
merged 1 commit into from
Oct 6, 2023

Conversation

daniel-weisse
Copy link
Member

@daniel-weisse daniel-weisse commented Oct 6, 2023

Context

As part of our image build pipeline, our API is updated to include measurements and a reference to the image in the API.
Currently, the image build may fail when calculating the expected measurements, but the API is still updated.
This means our API reports a image as being available, even though the build pipeline failed.
While this does not impact regular users, it impacts our E2E tests which automatically use the latest available image version and try to fetch measurements for that version.
If the measurements are not available, the test fails.

Proposed change(s)

  • Ensure the API is only updated if both the image upload (+image lookup table upload) and the measurements upload was successful

Additional info

Checklist

  • Update docs
  • Add labels (e.g., for changelog category)
  • Is PR title adequate for changelog?
  • Link to Milestone

@daniel-weisse daniel-weisse added the no changelog Change won't be listed in release changelog label Oct 6, 2023
@daniel-weisse daniel-weisse requested a review from malt3 October 6, 2023 07:41
@netlify
Copy link

netlify bot commented Oct 6, 2023

Deploy Preview for constellation-docs canceled.

Name Link
🔨 Latest commit bfbea62
🔍 Latest deploy log https://app.netlify.com/sites/constellation-docs/deploys/651fba230da3c0000877443a

@malt3
Copy link
Contributor

malt3 commented Oct 6, 2023

🏌️

@daniel-weisse daniel-weisse merged commit 8bb23c3 into main Oct 6, 2023
@daniel-weisse daniel-weisse deleted the fix/ci/premature-api-update branch October 6, 2023 12:34
msanft pushed a commit that referenced this pull request Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog Change won't be listed in release changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants