-
Notifications
You must be signed in to change notification settings - Fork 84
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
bug(zenko-699): Fix Lifecycle Test #182
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need some clarification on the new values file.
@@ -0,0 +1,75 @@ | |||
nodeCount: &nodeCount 1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this for local testing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just posting here the discussion Taylor and I had offline:
The concern was fundamentally changing an existing cluster when running this make command but it seems like it already previously had this intended affect. It's out of the scope of this PR but we should look into changing up the Makefile to better suit our needs.
image: | ||
pullPolicy: Always | ||
orbit: | ||
endpoint: "http://ciutil-orbit-simulator:4222" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If this is for local testing do we want orbit-simulator references?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
orbit sim still runs for local test runs
9235361
to
bde1e7f
Compare
No description provided.