Skip to content

[DOCS] Skip synced flush doc tests #49986

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

Merged
merged 1 commit into from
Dec 9, 2019
Merged

[DOCS] Skip synced flush doc tests #49986

merged 1 commit into from
Dec 9, 2019

Conversation

jrodewig
Copy link
Contributor

@jrodewig jrodewig commented Dec 9, 2019

The current snippets in the synced flush docs can cause conflicts with other background syncs, such as the global checkpoint sync or retention lease sync, in the docs tests.

This skips tests for those snippets to avoid conflicts.

Fixes #49623.

@jrodewig jrodewig added >docs General docs changes :Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. v8.0.0 v7.6.0 v7.5.1 labels Dec 9, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Distributed)

@jrodewig jrodewig requested review from debadair and ywelsch December 9, 2019 14:27
The current snippets in the synced flush docs can cause conflicts with
other background syncs, such as the global checkpoint sync or retention
lease sync, in the docs tests.

This skips tests for those snippets to avoid conflicts.
@jrodewig jrodewig changed the title [DOCS] Skip synced flush docs tests [DOCS] Skip synced flush doc tests Dec 9, 2019
Copy link
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

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

LGTM

@jrodewig jrodewig merged commit e520b85 into elastic:master Dec 9, 2019
@jrodewig jrodewig deleted the remove-synced-flush-doc-tests branch December 9, 2019 18:16
jrodewig added a commit that referenced this pull request Dec 9, 2019
The current snippets in the synced flush docs can cause conflicts with
other background syncs, such as the global checkpoint sync or retention
lease sync, in the docs tests.

This skips tests for those snippets to avoid conflicts.
jrodewig added a commit that referenced this pull request Dec 9, 2019
The current snippets in the synced flush docs can cause conflicts with
other background syncs, such as the global checkpoint sync or retention
lease sync, in the docs tests.

This skips tests for those snippets to avoid conflicts.
@jrodewig
Copy link
Contributor Author

jrodewig commented Dec 9, 2019

master: e520b85
7.x: 3359438
7.5: dab1b8e

SivagurunathanV pushed a commit to SivagurunathanV/elasticsearch that referenced this pull request Jan 23, 2020
The current snippets in the synced flush docs can cause conflicts with
other background syncs, such as the global checkpoint sync or retention
lease sync, in the docs tests.

This skips tests for those snippets to avoid conflicts.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Distributed Indexing/Distributed A catch all label for anything in the Distributed Indexing Area. Please avoid if you can. >docs General docs changes v7.5.1 v7.6.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] 409 Conflicts in synced-flush docs test
4 participants