-
Notifications
You must be signed in to change notification settings - Fork 63
Use shorter names in nightly tests #681
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
Conversation
jamOne-
left a comment
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.
These logs don't contain any credentials, right?
They contain only error messages returned by gcloud, I highly doubt that will print any credentials |
scaliby
left a comment
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.
As @jamOne- mentioned, please verify these logs doesn't contain any credentials. AFAIK they don't so approving.
Fix private cluster integration test by using shorter names.
Currently private cluster test always fails with:
ERROR: (gcloud.beta.container.node-pools.create) ResponseError: code=400, message=Node_pool.name must be less than 40 characters.
Some other changes include, uploading an artifact with the NodePools error log in cluster creating tests to improve debugging