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

Fix console pod is not starting in tests #1487

Merged
merged 1 commit into from
Mar 3, 2023

Conversation

pjuarezd
Copy link
Member

@pjuarezd pjuarezd commented Mar 3, 2023

In the operator tests under /testing/ the console pod do not start, that is because is trying to use the latest operator release v4.5.8

image: minio/operator:v4.5.8

This is to use the local built operator docker image for console instead of latest released operator container

@allanrogerr
Copy link
Contributor

gotest failing twice @pjuarezd - any connection?

@pjuarezd
Copy link
Member Author

pjuarezd commented Mar 3, 2023

gotest failing twice @pjuarezd - any connection?

Is not related, that test (gotest) failing was introduced in this PR merge #1469, se the action failing here https://github.com/minio/operator/actions/runs/4317695680/jobs/7535437371

I am trying to fix some of the tests that PR broke, but it's stuck in the gotest step, preventing to run the other kubernetes test.

@pjuarezd
Copy link
Member Author

pjuarezd commented Mar 3, 2023

@dvaldivia is planned to fix the failing tests? I see @cniackz has other PR #1486. Is related? I mean, many tests are broken in master as result of the merge of #1469

@dvaldivia
Copy link
Collaborator

Yes, I'll work on this as well as soon as I'm done testing the STS PR

reivaj05
reivaj05 previously approved these changes Mar 3, 2023
This is to use the local built operator docker image for console instead of latest released operator container
@pjuarezd pjuarezd merged commit 14fed14 into minio:master Mar 3, 2023
@pjuarezd pjuarezd deleted the fix-console-testing branch March 3, 2023 19:57
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.

4 participants