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

Use latest centos boxes instead of specific dates #353

Closed

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Jan 27, 2024

The centos/stream8 and centos/stream9 images are still not correct on Vagrant cloud, so it needs to be provided via a box_url. The centos/stream9 box URL is now a 404 so this changes it to latest.

The centos/stream8 and centos/stream9 images are still not correct on
Vagrant cloud, so it needs to be provided via a box_url. The
centos/stream9 box URL is now a 404 so this changes it to latest.
Copy link

codecov bot commented Jan 27, 2024

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (e67e9c1) 0.72% compared to head (87f2740) 0.72%.

Files Patch % Lines
lib/beaker-hostgenerator/data.rb 0.00% 2 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff           @@
##           master    #353   +/-   ##
======================================
  Coverage    0.72%   0.72%           
======================================
  Files          15      15           
  Lines        1802    1802           
======================================
  Hits           13      13           
  Misses       1789    1789           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@evgeni
Copy link
Member

evgeni commented Jan 27, 2024

Well, gotta update tests ;-)

@daberkow
Copy link
Contributor

This is needed for voxpupuli/puppet-selinux#390, I had to create another branch because I need a more up to date base branch, if this is merged in, we can just flip over to mater.

@bastelfreak
Copy link
Member

merged in #374

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants