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

CI fixes for v1 branch #451

Closed
8 tasks done
isc-tleavitt opened this issue Apr 12, 2024 · 2 comments
Closed
8 tasks done

CI fixes for v1 branch #451

isc-tleavitt opened this issue Apr 12, 2024 · 2 comments
Assignees
@isc-tleavitt @isc-shuliu
Labels
bug Something isn't working

Comments

@isc-tleavitt
Copy link
Contributor

isc-tleavitt commented Apr 12, 2024 •
edited
Loading

Splitting this out from #437 as a separate issue:

  • Be smarter about expired community edition licenses; could have a min version flag or a file with a list of tags to ignore? (need this in master and v1)
  • Fix failing compilation on IRIS for Health prior to 2024.1
  • Issues in the clean install step (this is likely to be the bigger can of worms, but we can at least investigate + fix the missing container part)
  • Fix unit tests (currently have a failure due to a code change) - we might need to talk about the code change
  • Fix "verify" phase (currently failing due to ZPM/IPM differences; see also zpm verify command fails with <CLASS DOES NOT EXIST> #428)
  • Ensure "verify" passes
  • Make test/verify failures fatal again
  • Really fix clean install + upgrade (this will require other work tracked under other issues, but zpm 0.7.0 unable to load zpm 0.9.0 (from v1-jan2024-updates) #449 was a good first step)
@isc-tleavitt isc-tleavitt added the bug Something isn't working label Apr 12, 2024
@isc-tleavitt isc-tleavitt self-assigned this Apr 12, 2024
@isc-tleavitt isc-tleavitt mentioned this issue Apr 12, 2024
Merged
isc-tleavitt added a commit that referenced this issue Apr 12, 2024
@isc-tleavitt
Run on fewer container versions
e9d6e93 
This will make CI faster and still provide sufficient coverage of relevant versions. We now only run on the major.minor container tags (latest maint release) and latest-preview. This skips over a bunch we were trying before that had expired community edition licenses.

This change partially addresses #451 but doesn't provide the entire solution.
@isc-tleavitt isc-tleavitt mentioned this issue Apr 12, 2024
Merged
@isc-tleavitt isc-tleavitt mentioned this issue Apr 12, 2024
Closed
@isc-tleavitt
Copy link
Contributor Author

#457 handles all but the clean install + upgrade.

This was referenced Apr 25, 2024
Merged
Merged
@isc-tleavitt
Copy link
Contributor Author

#466 incorporates a change to clean install + upgrade (assuming our upcoming v0.7.1 release as a basis)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@isc-tleavitt isc-tleavitt

@isc-shuliu isc-shuliu

Labels
bug Something isn't working
Projects
v0.9.0 release blockers
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@isc-tleavitt @isc-shuliu