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

Use data-dir and wal-dir to restore etcd snapshots #2548

Conversation

Zebradil
Copy link

@Zebradil Zebradil commented May 20, 2021

@superseb superseb self-requested a review May 20, 2021 13:37
@superseb
Copy link
Contributor

Thanks, will take a look

@Zebradil
Copy link
Author

Got some time to take a look this?

@Zebradil
Copy link
Author

@superseb is there a chance this will be merged?

@superseb
Copy link
Contributor

Can you make it 1 commit? And the only question I have on this why we are moving away from mv and now are using cp while we are still moving files around but just in subdirectories.

Copy link
Contributor

@superseb superseb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

^^

@Zebradil
Copy link
Author

Thanks for taking a look on this.

I don't remember already, why mv got changed to cp. There was a reason for that most likely. I'll check a version with mv to see why it didn't work.

@Zebradil Zebradil force-pushed the restore-snapshot-use-extra-args branch from 344c58e to 8dca30c Compare July 21, 2021 13:37
@Zebradil
Copy link
Author

@superseb sorry, just managed to get to this topic.

I replaced cp back to mv and also fixed another issue, which happens when old WAL files are not deleted. In this case etcd fails with error: "etcdmain: wal: crc mismatch". And squashed this to a single commit.

@Zebradil Zebradil requested a review from superseb July 30, 2021 09:33
@Zebradil
Copy link
Author

@superseb Any chance to have another look on that?

@superseb superseb added this to the RKE v1.3.2 - Rancher v2.6.2 milestone Aug 3, 2021
@superseb
Copy link
Contributor

@Zebradil CI is failing

@Zebradil Zebradil force-pushed the restore-snapshot-use-extra-args branch 2 times, most recently from 89a66a3 to 7ac48c2 Compare August 19, 2021 10:22
@Zebradil
Copy link
Author

@superseb I fixed formatting and squashed commits again. Now there is another error in CI, which doesn't seem actionable for me.

@superseb
Copy link
Contributor

@Zebradil Can you create a new PR against release/v1.3 so Drone picks it up for proper CI? Then I can merge it.

@Zebradil Zebradil changed the base branch from master to release/v1.3 November 20, 2021 14:10
@Zebradil Zebradil force-pushed the restore-snapshot-use-extra-args branch from 7ac48c2 to 058d1d5 Compare November 20, 2021 14:14
@Zebradil
Copy link
Author

@superseb done

@Zebradil
Copy link
Author

This is ridiculous, I'm tired updating my branch and resolving conflicts.

@Zebradil Zebradil closed this Dec 20, 2021
@Zebradil Zebradil deleted the restore-snapshot-use-extra-args branch December 20, 2021 09:23
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.

2 participants