-
Notifications
You must be signed in to change notification settings - Fork 22
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
Rename operations archive #116
Conversation
Also please check the test run https://github.com/ytsaurus/yt-k8s-operator/actions/runs/7829875497/job/21362727551?pr=116 — there are syntax errors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry that took so long, I missed that commits were added.
Looks good, could you please rebase on main branch and resolve conflict (I suppose it is scheduler > Scheduler
rename)?
Done |
one of e2es is failing, there is a chance that script is broken somehow |
Actually, the tests were green until I accidentally closed and reopened PR. Is it possible we have some flaps? |
i've restarted them, let's see if the issue stays. |
It may be unrelated (though persistent) flap, will look into that and merge after investigation. |
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en