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

To fix "Logs test on Kind" test by attempting more than one time getting the API response. #1007

Merged
merged 2 commits into from
Feb 7, 2022
Merged

Conversation

cniackz
Copy link
Contributor

@cniackz cniackz commented Feb 4, 2022

Hello Team,

Long story short, after changes in PR 989: operator:v4.4.4 tests become bit more inestable.
Nevertheless, when tested manually, noticed that Prometheus and Logs API were both reachable if we
attempted couple of times to get the response. The intention of this test, is to make these tests more
stable by adding a loop to re-try getting the answer from API until this is obtained as before.
Notice that previously, we just attempted this once.

Please give me feedback and if we should proceed this way or if you have any other idea,
I am happy to try other experiment.

Thanks!

@cniackz cniackz added the wip label Feb 4, 2022
@cniackz cniackz self-assigned this Feb 4, 2022
@cniackz cniackz changed the title [WIP] Perform more attempts to get API response Perform more attempts to get API response Feb 4, 2022
@cniackz cniackz removed the wip label Feb 4, 2022
@cniackz cniackz changed the title Perform more attempts to get API response To fix "Logs test on Kind" test by attempting more than one time getting the API response. Feb 4, 2022
@harshavardhana harshavardhana merged commit c8758c2 into minio:master Feb 7, 2022
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