-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
feature: sync .sealos workdir #3903
Conversation
🤖 Generated by lychee actionSummary
Full action output |
Codecov ReportPatch has no changes to coverable lines. 📢 Thoughts on this report? Let us know!. |
@@ -273,3 +268,30 @@ func (c *Applier) deleteCluster() error { | |||
logger.Info("succeeded in deleting current cluster") | |||
return nil | |||
} | |||
|
|||
func (c *Applier) syncWorkdir() { | |||
workDir := constants.ClusterDir(c.ClusterDesired.Name) |
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.
if exec sealos user is root and other node user name is not root , same this dir not using all nodes.
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.
yes !,it should be ~/. sealos/.
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.
if exec sealos user is root and other node user name is not root , same this dir not using all nodes.
I'll make some modifications to this code
/cherry-pick release-v4.4 |
🤖 says: cherry pick action finished successfully 🎉! |
Co-authored-by: 榴莲榴莲 <[email protected]>
support #3859