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

[deployer exec root-homes]: wait for the pod to be ready before exec-ing into it #5294

Closed
wants to merge 1 commit into from

Conversation

GeorgianaElena
Copy link
Member

Without the command is not working as the pod is not ready immediately

@GeorgianaElena GeorgianaElena added the deployer:skip-deploy Skips deployment of anything (support, staging, prod) label Dec 18, 2024

This comment was marked as off-topic.

@sgibson91
Copy link
Member

sgibson91 commented Dec 18, 2024

@sunu and I just found this this morning and he was going to fix it in another open PR 😆

Tarashish, is this matching what you were planning on implementing?

x-ref: #5291

@sgibson91
Copy link
Member

@GeorgianaElena We have a kubectl wait command incoming https://github.com/2i2c-org/infrastructure/pull/5291/files#diff-d91b6584fdc66b3801674ac83d7c37be3b3c4d231f93cef38bf1f6d3bc05f50aR228

Are you happy to close this in favour of that PR? Thank you for the work though!

@GeorgianaElena
Copy link
Member Author

Sure! Also, I now just realized the importance of opening issues when a bug is found rather than going ahead and try to fix it. (I meant to open one but thought it will be a quick fix 🙄 )

@sgibson91
Copy link
Member

I super-appreciate you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployer:skip-deploy Skips deployment of anything (support, staging, prod)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants