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

Use the new name for cassandra-statefulset.yaml in e2e tests. #36418

Merged
merged 1 commit into from Nov 9, 2016
Merged

Use the new name for cassandra-statefulset.yaml in e2e tests. #36418

merged 1 commit into from Nov 9, 2016

Conversation

jimmycuadra
Copy link
Contributor

@jimmycuadra jimmycuadra commented Nov 8, 2016

Fixes #36323.


This change is Reviewable

@k8s-ci-robot
Copy link
Contributor

Can a kubernetes member verify that this patch is reasonable to test? If so, please reply with "@k8s-bot ok to test" on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands will still work. Regular contributors should join the org to skip this step.

If you have questions or suggestions related to this bot's behavior, please file an issue against the kubernetes/test-infra repository.

@k8s-github-robot k8s-github-robot added size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. release-note-label-needed labels Nov 8, 2016
@jimmycuadra jimmycuadra mentioned this pull request Nov 8, 2016
11 tasks
@foxish foxish assigned foxish and unassigned fejta Nov 8, 2016
@foxish
Copy link
Contributor

foxish commented Nov 8, 2016

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Nov 8, 2016
@k8s-github-robot k8s-github-robot added the do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. label Nov 8, 2016
@k8s-ci-robot
Copy link
Contributor

Jenkins unit/integration failed for commit 057d9df. Full PR test history.

The magic incantation to run this job again is @k8s-bot unit test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@jimmycuadra
Copy link
Contributor Author

Test failure looks unrelated. Flake?

@foxish foxish added release-note-none Denotes a PR that doesn't merit a release note. and removed do-not-merge DEPRECATED. Indicates that a PR should not merge. Label can only be manually applied/removed. release-note-label-needed labels Nov 8, 2016
@foxish
Copy link
Contributor

foxish commented Nov 8, 2016

@k8s-bot unit test this

@foxish foxish added this to the v1.5 milestone Nov 8, 2016
@k8s-ci-robot
Copy link
Contributor

Jenkins GCE e2e failed for commit 057d9df. Full PR test history.

The magic incantation to run this job again is @k8s-bot cvm gce e2e test this. Please help us cut down flakes by linking to an open flake issue when you hit one in your PR.

@foxish
Copy link
Contributor

foxish commented Nov 8, 2016

@k8s-bot cvm gce e2e test this

@janetkuo janetkuo added the priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. label Nov 8, 2016
@janetkuo
Copy link
Member

janetkuo commented Nov 8, 2016

P0 for fixing flakes

@k8s-github-robot
Copy link

@k8s-bot test this [submit-queue is verifying that this PR is safe to merge]

@k8s-github-robot
Copy link

Automatic merge from submit-queue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm "Looks good to me", indicates that a PR is ready to be merged. priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. release-note-none Denotes a PR that doesn't merit a release note. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants