Skip to content
This repository has been archived by the owner on Nov 6, 2018. It is now read-only.

Updated apiVersion for SecurityContextConstraints #274

Merged
merged 2 commits into from
Jan 18, 2018

Conversation

piyush-garg
Copy link
Contributor

Added a case while generating SecurityContextConstraints
to set the default apiVersion to v1

This will also fix the SecurityContextConstraints problem in Kubernetes Client
#962

@oscerd
Copy link
Member

oscerd commented Jan 17, 2018

Ok to test

@oscerd
Copy link
Member

oscerd commented Jan 17, 2018

retest this please

@piyush-garg
Copy link
Contributor Author

@oscerd I think there is some problem with CI, it gives Java Heap Space- Out of Memory Error. But it locally gets build.

Copy link
Member

@hrishin hrishin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@piyush1594 this is a much better approach.
Thanks a lot!

@oscerd
Copy link
Member

oscerd commented Jan 18, 2018

retest this please

1 similar comment
@piyush-garg
Copy link
Contributor Author

retest this please

@oscerd
Copy link
Member

oscerd commented Jan 18, 2018

Van you rebase this one too?

@oscerd
Copy link
Member

oscerd commented Jan 18, 2018

It seems that retest doesn't work. I tried to replay the build from UI bit it still take the old heap setting

@piyush-garg
Copy link
Contributor Author

Rebase done, let's see
Thanks @oscerd

@oscerd oscerd merged commit d96f6a1 into fabric8io:master Jan 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants