-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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: bump VMs #21129
CI: bump VMs #21129
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: edsantiago The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
LGTM |
The debian failure is real; I've just pushed a new VM build to work around it. |
All VMs have pasta 2023-12-04, so, remove a skip. Signed-off-by: Ed Santiago <[email protected]>
/hold @edsantiago giving you the LGTM assuming it finishes, you can drop the hold. |
Thanks @baude I have a memory that there were other tests being skipped, waiting for *vark-1.9. We now have 1.9.0-1 on all Fedoras, but (sigh) I can't remember what those skips were nor can I find any documentation or hints. Does anyone remember what I'm talking about? |
Nope, perhaps @Luap99 does. |
I am not aware of anything specific, there is a netavark-firewalld-reload.service but that needs a running firewalld which AFAIK is not the case in our CI env and it is already covered by netavark CI tests so I don't think it is strictly required to tests it here. |
Thanks. /hold cancel |
490e5ed
into
containers:main
All VMs have pasta 2023-12-04, so, remove a skip.
Signed-off-by: Ed Santiago [email protected]