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

[BUG] org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=nodes.stats/30_discovery/Discovery stats} #10931

Closed
shwetathareja opened this issue Oct 25, 2023 · 5 comments
Assignees
Labels
bug Something isn't working untriaged

Comments

@shwetathareja
Copy link
Member

Describe the bug
Discovery stats structure got updated in #10670
This is causing org.opensearch.backwards.MixedClusterClientYamlTestSuiteIT.test {p0=nodes.stats/30_discovery/Discovery stats} tests to fail

https://build.ci.opensearch.org/job/gradle-check/28987/

@shwetathareja shwetathareja added bug Something isn't working untriaged labels Oct 25, 2023
@shwetathareja
Copy link
Member Author

@amkhar can you please take a look and update the test asap.

@amkhar
Copy link
Contributor

amkhar commented Oct 26, 2023

I'll take a look shortly. Thanks for reporting this.

@amkhar
Copy link
Contributor

amkhar commented Oct 26, 2023

These mixed cluster test were failing even before https://build.ci.opensearch.org/job/gradle-check/28795/testReport/junit/org.opensearch.b[…]iteIT/test__p0_nodes_stats_30_discovery_Discovery_stats_/
GC link : https://build.ci.opensearch.org/job/gradle-check/28795/
Also I checked the code specifically for discovery one

- do:
      nodes.stats:
        filter_path: "nodes.*.discovery"

  - is_false: cluster_name
  - is_true:  nodes
  - is_false: nodes.$master.name
  - is_false: nodes.$master.jvm
  - is_true:  nodes.$master.discovery

I did this from code base.

curl localhost:9200/_nodes/stats?filter_path=nodes.*.discovery
{"nodes":{"Ubn8Zxo9R06G299on2XfCw":{"discovery":{"cluster_state_queue":{"total":0,"pending":0,"committed":0},"published_cluster_states":{"full_states":2,"incompatible_diffs":0,"compatible_diffs":0},"cluster_state_stats":{"overall":{"update_count":2,"total_time_in_millis":101,"failed_count":0}}}}}}

We’ll always get discovery section as nothing is changed in that.

After rebasing from main these failures will be gone as new version which is merged in main is 2.12.0
Please close this issue once you verify that other PRs are not facing this after merging from main.

@shwetathareja
Copy link
Member Author

Closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working untriaged
Projects
None yet
Development

No branches or pull requests

2 participants