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

Upgrade templates to use apps/v1 schema, also fixes a build problem #79

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

falken
Copy link
Contributor

@falken falken commented Nov 8, 2019

No description provided.

@tiwarishrijan
Copy link

Hi @falken , Any update on your fixes. As namenode and zookeper are not working.

@falken
Copy link
Contributor Author

falken commented Dec 7, 2019

Are they not working in my branch? Or on master?

@tiwarishrijan
Copy link

Are they not working in my branch? Or on master?

Tried both Master branch and your branch, none of them seems working.

@tiwarishrijan
Copy link

@falken I guess this is still failing any fix on the same ?
image

@falken
Copy link
Contributor Author

falken commented Jan 13, 2020

@tiwarishrijan I did what I could, but the build environment doesn't support this build. I was unable to perform the tests locally either to verify the change. I do know that my fork does work for me.

The build hasn't actually passed on Master in at least 3 months, so I'm not sure it's a good indicator

Copy link

@bayudwiyansatria bayudwiyansatria left a comment

Choose a reason for hiding this comment

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

kubernetes 1.6 or higher work with apps/v1

Copy link

@sjgiri sjgiri left a comment

Choose a reason for hiding this comment

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

Thanks!

@oneslideicywater
Copy link

does this repo in active maintainence?

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.

5 participants