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

Temporarily disable CentOS 8 tests in Molecule #1116

Merged
merged 1 commit into from
Feb 23, 2022

Conversation

ajdecon
Copy link
Collaborator

@ajdecon ajdecon commented Feb 23, 2022

Starting on Jan 31, the CentOS 8 mirror lists are no longer working. This
is causing our CentOS 8 tests in Molecule to fail because we can't
install packages.

While I've tested using the CentOS Vault instead of the canonical mirror
list, this isn't a great solution:

  • It's a bit on the slow side, because we don't have a global network of
    mirrors
  • CentOS 8 is no longer supported, so we'd just be targeting an
    increasingly old distribution

We're discussing how to address this in our automated tests. But for
now, we'll disable these tests so that our other Molecule tests will
run.

Starting on Jan 31, the CentOS 8 mirror lists are no longer working. This
is causing our CentOS 8 tests in Molecule to fail because we can't
install packages.

While I've tested using the CentOS Vault instead of the canonical mirror
list, this isn't a great solution:

- It's a bit on the slow side, because we don't have a global network of
mirrors
- CentOS 8 is no longer supported, so we'd just be targeting an
increasingly old distribution

We're discussing how to address this in our automated tests. But for
now, we'll disable these tests so that our other Molecule tests will
run.
@ajdecon ajdecon requested a review from dholt February 23, 2022 20:02
@ajdecon ajdecon merged commit 12a6a43 into NVIDIA:master Feb 23, 2022
@supertetelman
Copy link
Collaborator

I am also disabling the tests in Jenkins.

@ajdecon ajdecon mentioned this pull request Apr 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants