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

Add RHEL/CentOS 8 to list of RHEL flavors #11453

Closed
wants to merge 1 commit into from

Conversation

dustymabe
Copy link
Contributor

This will make it so that :rhel_8 will be returned on el8
guests.

This will make it so that `:rhel_8` will be returned on el8
guests.
@hashicorp-cla
Copy link

hashicorp-cla commented Mar 15, 2020

CLA assistant check
All committers have signed the CLA.

dustymabe added a commit to dustymabe/vagrant-sshfs that referenced this pull request Mar 15, 2020
See hashicorp/vagrant#11453. Here we add a
compatibility function to handle :rhel_8 because otherwise we'd have
to wait until a new version of vagrant got released and rolled out
everywhere before our users could take advantage of this.
@jbonhag
Copy link
Contributor

jbonhag commented Mar 16, 2020

Hi @dustymabe, thanks for opening the PR! This looks fine, but I have two requests/suggestions:

  1. Could you please add some examples of the :rhel_8 flavors to test/unit/plugins/guests/redhat/cap/flavor_test.rb?
  2. We have a :rhel_7 override for installing Docker in plugins/provisioners/docker/cap/redhat/docker_install.rb -- I assuming that :rhel_8 should behave like :rhel_7, so would you be willing to add that code?

Thanks again for the PR! 🎉

@jbonhag jbonhag self-requested a review March 17, 2020 14:25
@soapy1
Copy link
Contributor

soapy1 commented Apr 13, 2020

@dustymabe thank you for the PR! This has been superseded as part of the work in #11462

@soapy1 soapy1 closed this Apr 13, 2020
@dustymabe
Copy link
Contributor Author

Thanks sorry I didn't have time to get back to this PR

@ghost
Copy link

ghost commented May 14, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators May 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants