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.

Sign up for GitHub

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

Jump to bottom

Temporarily disable TestReversalStatefulSetRename #3845

Merged
merged 1 commit into from
Oct 15, 2020
Merged

Temporarily disable TestReversalStatefulSetRename #3845

merged 1 commit into from
Oct 15, 2020

Conversation

sebgl
Copy link
Contributor

@sebgl sebgl commented Oct 15, 2020

We have identified the root cause(s). Let's disable this test until
we properly fix it in the next few days.

Relates #3844.

@sebgl
Temporarily disable TestReversalStatefulSetRename
2ef9a70 
We have identified the root cause(s). Let's disable this test until
we properly fix it in the next few days.
pebrc
pebrc approved these changes Oct 15, 2020
View reviewed changes
test/e2e/es/reversal_test.go
RunESMutationReversal(t, b, renamed)
}
// TODO: re-enable once https://github.com/elastic/cloud-on-k8s/issues/3844 is fixed.
//func TestReversalStatefulSetRename(t *testing.T) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would have done t.SkipNow() I guess but this works too.

@sebgl
Copy link
Contributor Author

sebgl commented Oct 15, 2020

Jenkins test this please.

1 similar comment
@sebgl
Copy link
Contributor Author

sebgl commented Oct 15, 2020

Jenkins test this please.

@sebgl
Copy link
Contributor Author

sebgl commented Oct 15, 2020

Ci fails for some reason:

6:52:41  Using context: devops-ci
16:52:42  hudson.plugins.git.GitException: Command "git fetch --tags --progress --prune -- origin +refs/heads/${sha1}:refs/remotes/origin/${sha1}" returned status code 128:
16:52:42  stdout: 
16:52:42  stderr: fatal: Couldn't find remote ref refs/heads/${sha1}

I'm using my superpowers to merge this.

@sebgl sebgl merged commit c77b167 into elastic:master Oct 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pebrc pebrc pebrc approved these changes

Assignees
No one assigned
Labels
>flaky_test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sebgl @pebrc