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

Update: change supported images #49

Merged
merged 4 commits into from
Oct 5, 2024

Conversation

BenjamenMeyer
Copy link
Member

Add Distros

  • Ubuntu (Noble)
  • Linux Mint (21.2, 21.3, 22)
  • OpenSuSe (15.6)
  • Rocky Linux (8.10, 9.4)

Drop Distros

  • Ubuntu (mantic, bionic)
  • Debian (buster)
  • Fedora (34, 35, 36, 37, 38)
  • Rocky Linux (8.5, 8.6, 8.7, 8.8, 8.9, 9.0, 9.1, 9.2, 9.3)

@BenjamenMeyer BenjamenMeyer added the enhancement New feature or request label Oct 5, 2024
@BenjamenMeyer BenjamenMeyer self-assigned this Oct 5, 2024
Copy link

@royfalk royfalk left a comment

Choose a reason for hiding this comment

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

Looks fine. How do we test these new versions? Are we using github CICD to check they compile?

- 'opensuse/leap:15.5'
- 'opensuse/leap:15.4'
- 'opensuse/leap:15.3'
- 'opensuse/leap:15.2'
#- 'fedora:41'
- 'fedora:40'
- 'fedora:39'
Copy link

Choose a reason for hiding this comment

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

Can we do this with a variable? Something like opensuse/leap:>15.2?

Copy link
Member Author

Choose a reason for hiding this comment

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

These directly translate to Docker Image names so no we can't use comparisons, etc.

@@ -124,16 +124,43 @@ function bootstrapOnDebian()
lsb-release
;;
"buster")
echo "Sorry, Debian buster is no longer supported"
Copy link

Choose a reason for hiding this comment

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

We can probably do an array of unsupported and a for/each.

Copy link
Member Author

Choose a reason for hiding this comment

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

Its more for helping us catch what we deprecated. We can clean them up to reduce the list after a time.

@BenjamenMeyer
Copy link
Member Author

Looks fine. How do we test these new versions? Are we using github CICD to check they compile?

We use GitHub Actions to see that the images build successfully. Generally this works fine, but when there's a major distro change we might have to rework dependencies but we won't catch that until we're using the images in the engine repo right now.

Not sure we need to check that in the GHA here. I have a modified version of one of the scripts for doing local image builds to test and help diagnose failures there. I could probably add a second for running with the engine build too as a local test thing.

We do want to keep the effort here relatively light.

@BenjamenMeyer BenjamenMeyer merged commit 6f750d3 into vegastrike:master Oct 5, 2024
49 of 51 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants