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

support OKD4 #145

Closed
walidshaari opened this issue Aug 16, 2020 · 7 comments
Closed

support OKD4 #145

walidshaari opened this issue Aug 16, 2020 · 7 comments
Labels
question/support This is not a bug but a question or support

Comments

@walidshaari
Copy link
Contributor

the current repo has many settings which are wired to ocp4, it would be nice if we can expand it to OKD4

@christianh814 christianh814 added the question/support This is not a bug but a question or support label Aug 17, 2020
@christianh814
Copy link
Contributor

Which sections, that cannot be altered by a vars.yaml file is hardcoded to OCP4? The playbook should work with OKD4 artifacts.

@walidshaari
Copy link
Contributor Author

I see your point as long as we can override them as variables, yes, it should be fine or add them in the vars examples, I see one example that is close https://github.com/RedHatOfficial/ocp4-helpernode/blob/master/docs/examples/vars-nightlies.yaml, will have to test it.

@christianh814
Copy link
Contributor

Good point, it would be nice to at least have an example. If you could get a sample vars.yaml that works, I can include it in the examples

@christianh814
Copy link
Contributor

@walidshaari Actually, it might not work. see #134 ...the new version of RHCOS will now require a new artefact that the helper doesn't account for currently.

As it stands, the helper will not work with OKD until version 4.6 comes out. I don't want to really make changes with a lot of conditionals if we just need to remove them for 4.6.

I'll leave this open and will come back this this once OCP v4.6 is GA.

@christianh814
Copy link
Contributor

..well...maybe not coreos/fedora-coreos-tracker#390 (comment)

@walidshaari
Copy link
Contributor Author

let me know if there is a task I can take.

@christianh814
Copy link
Contributor

@walidshaari Please do! I would verify to see if it works "as is"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question/support This is not a bug but a question or support
Projects
None yet
Development

No branches or pull requests

2 participants