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.

Sign up for GitHub

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

Jump to bottom

Add upgrade test for unicorn flavor #577

Closed
mjnagel opened this issue Jul 15, 2024 · 0 comments · Fixed by #585
Closed

Add upgrade test for unicorn flavor #577

mjnagel opened this issue Jul 15, 2024 · 0 comments · Fixed by #585
Assignees
@mjnagel
Labels
ci Issues pertaining to CI / Pipelines / Testing enhancement New feature or request

Comments

@mjnagel
Copy link
Contributor

mjnagel commented Jul 15, 2024

Is your feature request related to a problem? Please describe.

Our initial unicorn flavor addition did not have the upgrade test because there was no previous version published. Currently CI does not test upgrades of the unicorn flavor.

Describe the solution you'd like

When a PR is pushed up it should be run through install/upgrade tests for all flavors.

Additional context

This will require adjustments in a few places:
@mjnagel mjnagel added enhancement New feature or request ci Issues pertaining to CI / Pipelines / Testing labels Jul 15, 2024
@mjnagel mjnagel self-assigned this Jul 16, 2024
@mjnagel mjnagel mentioned this issue Jul 17, 2024
Merged
5 tasks
mjnagel added a commit that referenced this issue Jul 19, 2024
@mjnagel
ci: add unicorn upgrade test (#585)
30cbae4 
## Description

Adds the `unicorn` flavor upgrade test. To make the
`latest-package-release` lookup from the correct repository (`/private`
if `unicorn`) I added a utils task. I also modified the publish to use
this same utils task for consistency.

Tested publish workflow changes on fork (excluded create, so expected
failures but output shows the correct repositories and zarf packages
being referenced):
- Snapshot: https://github.com/BagelLab/uds-core/actions/runs/9981790347
- Normal: https://github.com/BagelLab/uds-core/actions/runs/9981855590

## Related Issue

Fixes #577

## Type of change

- [ ] Bug fix (non-breaking change which fixes an issue)
- [ ] New feature (non-breaking change which adds functionality)
- [x] Other (security config, docs update, etc)

## Checklist before merging

- [x] Test, docs, adr added or updated as needed
- [x] [Contributor
Guide](https://github.com/defenseunicorns/uds-template-capability/blob/main/CONTRIBUTING.md)
followed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mjnagel mjnagel

Labels
ci Issues pertaining to CI / Pipelines / Testing enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ci: add unicorn upgrade test defenseunicorns/uds-core
1 participant
@mjnagel