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.

Sign up for GitHub

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

Jump to bottom

Run full update only for master rebuild #158

Closed
l0kix2 opened this issue Mar 8, 2024 · 3 comments · Fixed by #164
Closed

Run full update only for master rebuild #158

l0kix2 opened this issue Mar 8, 2024 · 3 comments · Fixed by #164
Assignees
@qurname2
Labels
backlog Backlog

Comments

@l0kix2
Copy link
Collaborator

l0kix2 commented Mar 8, 2024

Currently it also happens for data nodes and tablet nodes
@chizhonkova chizhonkova added the backlog Backlog label Mar 13, 2024
@qurname2 qurname2 mentioned this issue Mar 18, 2024
Merged
@qurname2 qurname2 self-assigned this Mar 18, 2024
@l0kix2 l0kix2 reopened this Mar 18, 2024
@l0kix2
Copy link
Collaborator Author

l0kix2 commented Mar 18, 2024

Tablet cells better be updated with tablet cells deletion/recreation step (which is currently happens in full update), i suggest we incapsulate it inside tablet nodes flow later.

@qurname2
Copy link
Collaborator

PR for data-node only: #166

@l0kix2
Copy link
Collaborator Author

l0kix2 commented Apr 16, 2024

This should be not needed with #115 and #211

@l0kix2 l0kix2 closed this as not planned Won't fix, can't repro, duplicate, stale Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qurname2 qurname2

Labels
backlog Backlog
Projects
yt-k8s-operator project
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

replaced NeedFullUpdate in favour of NeedLocalUpdate for data,tablet-… qurname2/yt-k8s-operator
3 participants
@l0kix2 @qurname2 @chizhonkova