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

[Platform] Do not use sudo in yb_backup.py script if possible #5440

Closed
WesleyW opened this issue Aug 19, 2020 · 0 comments
Closed

[Platform] Do not use sudo in yb_backup.py script if possible #5440

WesleyW opened this issue Aug 19, 2020 · 0 comments
Assignees
@WesleyW
Labels
area/platform Yugabyte Platform
Projects
Platform

Comments

@WesleyW
Copy link
Contributor

WesleyW commented Aug 19, 2020

When running the yb_backup.py script, do not run "sudo su yugabyte" if possible. This means defaulting to ssh'ing as yugabyte user and removing the sudo commands.
@WesleyW WesleyW added the area/platform Yugabyte Platform label Aug 19, 2020
@WesleyW WesleyW self-assigned this Aug 19, 2020
@streddy-yb streddy-yb added this to Done in Platform Aug 25, 2020
WesleyW added a commit that referenced this issue Aug 27, 2020
@WesleyW
#5440: Fix missing backup argument
c7f7895 
Summary: D9237 did not have correct arguments for backup function. THis fixes it.

Test Plan: backup

Reviewers: bogdan, daniel, arnav

Reviewed By: arnav

Subscribers: jenkins-bot, yugaware

Differential Revision: https://phabricator.dev.yugabyte.com/D9257
WesleyW added a commit that referenced this issue Aug 27, 2020
@WesleyW
#5440: Fix missing backup argument
49f61b2 
Summary: D9237 did not have correct arguments for backup function. THis fixes it.

Test Plan: backup

Reviewers: bogdan, daniel, arnav

Reviewed By: arnav

Subscribers: jenkins-bot, yugaware

Differential Revision: https://phabricator.dev.yugabyte.com/D9257
WesleyW added a commit that referenced this issue Aug 27, 2020
@WesleyW
#5440: Fix missing backup argument
5fa528e 
Summary: D9237 did not have correct arguments for backup function. THis fixes it.

Test Plan: backup

Reviewers: bogdan, daniel, arnav

Reviewed By: arnav

Subscribers: jenkins-bot, yugaware

Differential Revision: https://phabricator.dev.yugabyte.com/D9257
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@WesleyW WesleyW

Labels
area/platform Yugabyte Platform
Projects
Platform
  
Closed
Milestone
No milestone
Development

No branches or pull requests
2 participants
@WesleyW @streddy-yb