Skip to content

Openshift Updates #112

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

Closed
wants to merge 6 commits into from
Closed

Conversation

automatecloud
Copy link
Contributor

@automatecloud automatecloud commented Jul 7, 2020

Fixed the following:

  1. Better structure of the Installation flow with integration of YAML and Helm Chart in one flow and not different chapters.
  2. Added all necessary changes for Openshift 4 where CRI-O is a must.
  3. Added the necessary changes for the Defenders if they are deployed as a helm chart in Openshift 4.
  4. Added PSP example to the installation of Kubernetes

Andreas Wilke and others added 6 commits May 6, 2020 16:01
Adding examples to the Open Policy Agent policies.
Fixed the following:
1. Better structure of the Installation flow with integration of YAML and Helm Chart in one flow and not different chapters.
2. Added all necessary changes for Openshift 4 where CRI-O is a must.
3. Added the necessary changes for the Defenders if they are deployed as a helm chart in Openshift 4.
@ryanchauvin
Copy link
Contributor

@iansk this is ready to merge

@ryanchauvin
Copy link
Contributor

@iansk could you please look at merging this one as well? It's showing a conflict, not sure how to fix that.

@sgordon46
Copy link
Collaborator

@automatecloud there is a merge conflict with this one. Can you fix?

@automatecloud
Copy link
Contributor Author

@sgordon46 i did a complete rewriting of the openshift installation back in May - July that should be merged before we merge all other openshift updates if you ask me? @iansk what can we do now?

@ryanchauvin
Copy link
Contributor

@automatecloud do you have a PR for those changes?

@automatecloud
Copy link
Contributor Author

@ryanchauvin no but i think we can use yours ? #148

@iansk
Copy link
Collaborator

iansk commented Nov 12, 2020

Excellent work @automatecloud. I'm merging a variation of your work in this PR: #239

Also, thanks @ryanchauvin for additional follow-on fixes.

Closing unmerged.

@iansk iansk closed this Nov 12, 2020
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.

4 participants