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

PMM-12712 Shards collector. #762

Merged

Conversation

JiriCtvrtka
Copy link
Contributor

@JiriCtvrtka JiriCtvrtka commented Dec 13, 2023

main.go Outdated Show resolved Hide resolved
exporter/topology_info_test.go Outdated Show resolved Hide resolved
exporter/sharded_collector.go Outdated Show resolved Hide resolved
exporter/sharded_collector.go Outdated Show resolved Hide resolved
exporter/topology_info.go Outdated Show resolved Hide resolved
exporter/topology_info.go Outdated Show resolved Hide resolved
exporter/topology_info.go Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
@percona percona deleted a comment from github-actions bot Dec 14, 2023
@JiriCtvrtka
Copy link
Contributor Author

I am trying to figure out why tests failing, but localy it is fine.

@JiriCtvrtka
Copy link
Contributor Author

JiriCtvrtka commented Jan 8, 2024

Tests stability is still not solved. Please review code in meanwhile.

@JiriCtvrtka JiriCtvrtka marked this pull request as ready for review January 8, 2024 09:11
@JiriCtvrtka JiriCtvrtka requested a review from a team as a code owner January 8, 2024 09:11
docker/scripts/init-shard.sh Outdated Show resolved Hide resolved
exporter/sharded_collector.go Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
exporter/shards_collector.go Outdated Show resolved Hide resolved
@idoqo idoqo self-requested a review January 8, 2024 12:13
@JiriCtvrtka JiriCtvrtka changed the title PMM-12712 Sharded collector. PMM-12712 Shards collector. Jan 9, 2024
@JiriCtvrtka JiriCtvrtka merged commit 0764ec6 into PMM-12510-correctly-recognize-arbiter-nodes Jan 15, 2024
7 checks passed
@JiriCtvrtka JiriCtvrtka deleted the PMM-12712-shards-info branch January 15, 2024 08:57
idoqo added a commit that referenced this pull request Jan 17, 2024
* add networks

* accept exit code

* use replica set status for node states

* drop external links

* fix formatting

* convert to table tests

* fix topology tests

* include hello command for arbiter nodes

* use parallel tests

* add tests for hello metrics

* attempt to satisfy golint

* use single command for replica setup

* improve naming

* reduce external calls

* add method comments

* use isMaster for compatibility

* PMM-12712 Shards collector. (#762)

* PMM-12712 Shard ID and count.

* PMM-12712 Name and count.

* PMM-12712 Sharded collector.

* PMM-12712 Remove missed print.

* PMM-12712 Revert test changes.

* PMM-12712 Revert old not related changes.

* PMM-12712 Another changes.

* PMM-12712 Missed print.

* PMM-12712 Align naming with task description.

* PMM-12712 Add mongos test client.

* PMM-12712 Lint.

* PMM-12712 Lint.

* PMM-12712 Typo.

* PMM-12712 Lint.

* PMM-12712 Test higher sleep.

* Revert "PMM-12712 Test higher sleep."

This reverts commit 88451ba.

* PMM-12712 Test of pipeline.

* PMM-12712 Change.

* PMM-12712 Test.

* PMM-12712 Test.

* PMM-12712 Test.

* PMM-12712 Correct aggregation to get chunks info.

* PMM-12712 Another progress.

* PMM-12702 Small refactor.

* PMM-12712 Test.

* PMM-12712 Fix, tests.

* PMM-12712 Temp.

* PMM-12712 Improve sharded test.

* PMM-12712 Change in workflow.

* PMM-12712 Remove print.

* PMM-12712 Init script changes.

* PMM-12712 Remove print.

* PMM-12712 Bigger sleep.

* PMM-12712 Refactor.

* PMM-12712 Small refactor.

* PMM-12712 Another refactor.

* PMM-12712 Fix after refactor.

* PMM-12712 Alias.

* PMM-12712 Alias for shell in older versions.

* PMM-12712 Static test shard.

* PMM-12712 Fix another tests, mongo 4 shell script.

* PMM-12712 Big sleep test.

* PMM-12712 Michael's fix.

* PMM-12712 Sleep.

* PMM-12712 Env.

* PMM-12712 Test.

* PMM-12712 Fix test.

* PMM-12712 Bigger sleep after changes.

* PMM-12712 Remove duplicate lines.

* PMM-12712 Renaming from sharded to shards.

* PMM-12712 Better script to detect proper mongo client.

* PMM-12712 Init script refactor.

* PMM-12712 Fix.

* PMM-12712 Small refactor.

* Update main.go

Co-authored-by: Michael Okoko <[email protected]>

* Update exporter/shards_collector.go

Co-authored-by: Michael Okoko <[email protected]>

* PMM-12712 Remove mongo 6 for now, skip test for shards.

* PMM-12712 Revert sleep length.

* PMM-12712 Mongo client based on vendor.

---------

Co-authored-by: Michael Okoko <[email protected]>

* add wait to docker compose command

* remove wait from docker compose command

---------

Co-authored-by: Jiří Čtvrtka <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants