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

Integ Test for RemotePublication can be disabled by rolling restart of masters #15677

Merged
merged 4 commits into from
Sep 5, 2024

Conversation

shiv0408
Copy link
Member

@shiv0408 shiv0408 commented Sep 4, 2024

Description

Integ Test for RemotePublication can be disabled by rolling restart of masters

To Disable the Remote Publication, disable the remote publication flag on the nodes and do a rolling restart of the master nodes.

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Copy link
Contributor

github-actions bot commented Sep 4, 2024

❌ Gradle check result for 9ffab92: null

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Sep 4, 2024

❌ Gradle check result for 9ffab92: null

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Sep 4, 2024

❌ Gradle check result for e39e17a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Sep 4, 2024

❌ Gradle check result for e39e17a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Sep 5, 2024

❌ Gradle check result for e39e17a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@shwetathareja
Copy link
Member

@shiv0408 rebase from main

Copy link
Contributor

github-actions bot commented Sep 5, 2024

❌ Gradle check result for c5920cf: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@shwetathareja shwetathareja added backport 2.x Backport to 2.x branch backport 2.17 labels Sep 5, 2024
Copy link
Contributor

github-actions bot commented Sep 5, 2024

❌ Gradle check result for c5920cf: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Sep 5, 2024

✅ Gradle check result for fad6eaf: SUCCESS

Copy link

codecov bot commented Sep 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.84%. Comparing base (729e40d) to head (fad6eaf).
Report is 12 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main   #15677      +/-   ##
============================================
- Coverage     71.95%   71.84%   -0.12%     
+ Complexity    64192    64147      -45     
============================================
  Files          5270     5271       +1     
  Lines        300052   300166     +114     
  Branches      43368    43380      +12     
============================================
- Hits         215917   215647     -270     
- Misses        66442    66822     +380     
- Partials      17693    17697       +4     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@shwetathareja shwetathareja merged commit 2eb148c into opensearch-project:main Sep 5, 2024
38 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 5, 2024
…f masters (#15677)

Signed-off-by: Shivansh Arora <[email protected]>
(cherry picked from commit 2eb148c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 5, 2024
…f masters (#15677)

Signed-off-by: Shivansh Arora <[email protected]>
(cherry picked from commit 2eb148c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@shiv0408 shiv0408 deleted the publish-disable-it branch September 5, 2024 18:13
akolarkunnu pushed a commit to akolarkunnu/OpenSearch that referenced this pull request Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.x Backport to 2.x branch skip-changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants