Skip to content

Commit

Permalink
Color paasta rollback warnings yellow instead of red (#3925)
Browse files Browse the repository at this point in the history
I've noticed a couple folks think that paasta rollback failed because of
this coloring - let's avoid some confusion and color this text yellow
instead (which is more accurate anyway since it's a warning)
  • Loading branch information
nemacysts authored Jul 30, 2024
1 parent 10b03bc commit d83bb8f
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions paasta_tools/cli/cmds/rollback.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,10 +57,10 @@ def add_subparser(subparsers: argparse._SubParsersAction) -> None:
epilog=(
"This rollback command uses the Git control plane, which requires network "
"connectivity as well as authorization to the Git repo.\n\n"
+ PaastaColors.red(
+ PaastaColors.yellow(
"WARNING: You MUST manually revert changes in Git and go through the normal push process after using this command.\n"
)
+ PaastaColors.red(
+ PaastaColors.yellow(
"WARNING: Failing to do so means that Jenkins will redeploy the latest code on the next scheduled build!"
)
),
Expand Down Expand Up @@ -334,12 +334,12 @@ def paasta_rollback(args: argparse.Namespace) -> int:

if returncode == 0:
print(
PaastaColors.red(
PaastaColors.yellow(
f"WARNING: You MUST manually revert changes in Git! Use 'git revert {rolled_back_from.sha}', and go through the normal push process. "
)
)
print(
PaastaColors.red(
PaastaColors.yellow(
f"WARNING: Failing to do so means that Jenkins will redeploy the latest code on the next scheduled build!"
)
)
Expand Down

0 comments on commit d83bb8f

Please sign in to comment.