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

Pin GHA runners to ubuntu-24.04 #1853

Merged
merged 3 commits into from
Sep 23, 2024
Merged

Pin GHA runners to ubuntu-24.04 #1853

merged 3 commits into from
Sep 23, 2024

Conversation

Molter73
Copy link
Collaborator

@Molter73 Molter73 commented Sep 18, 2024

Description

Having runners configured to ubuntu-latest means that the underlying VM could be updated to a newer version without us knowing. This should only be a problem when a new LTS version of Ubuntu is made the default for GHA and shouldn't be a major problem for us, since we mostly use generic tools pre-installed on all runners but, just in case, we should be explicit about what version of runner we want to use.

Checklist

  • Investigated and inspected CI test results
  • Updated documentation accordingly

Automated testing

  • Added unit tests
  • Added integration tests
  • Added regression tests

If any of these don't apply, please comment below.

Testing Performed

Run as many workflows as possible and checked none of them is broken.

@Molter73 Molter73 added run-cpaas-steps Trigger CI steps related to cPaaS run-multiarch-builds Run steps for non-x86 archs. and removed run-multiarch-builds Run steps for non-x86 archs. labels Sep 18, 2024
@Molter73 Molter73 changed the title Change all runners to ubuntu-24.04 and see what breaks Pin GHA runners to ubuntu-24.04 Sep 20, 2024
@Molter73 Molter73 marked this pull request as ready for review September 20, 2024 13:51
@Molter73 Molter73 requested a review from a team as a code owner September 20, 2024 13:51
Copy link
Collaborator

@Stringy Stringy left a comment

Choose a reason for hiding this comment

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

One minor question about the logging, but otherwise LGTM!

ansible/Makefile Outdated
@@ -29,7 +29,7 @@ provision-vms:

.PHONY: destroy-vms
destroy-vms:
ansible-playbook -i $(CONTEXT) \
ansible-playbook -i $(CONTEXT) -vvvv \
Copy link
Collaborator

Choose a reason for hiding this comment

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

What's the purpose of the extra logging?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I did this because of a weird flake I got, but then the issue never happened again and I forgot to remove the verbosity, I'll roll it back before merging.

@Molter73 Molter73 merged commit 7d6a3ed into master Sep 23, 2024
66 of 69 checks passed
@Molter73 Molter73 deleted the mauro/ubuntu-24.04 branch September 23, 2024 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
run-cpaas-steps Trigger CI steps related to cPaaS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants