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

Sync 3.11-main into master #1164

Merged
merged 437 commits into from
Nov 30, 2023
Merged

Sync 3.11-main into master #1164

merged 437 commits into from
Nov 30, 2023

Conversation

amildahl
Copy link
Contributor

@amildahl amildahl commented Nov 28, 2023

Sync backup-utils/3.11-main to backup-utils/master

jianghao0718 and others added 30 commits September 15, 2023 16:22
Add integration tests to backup-utils-private
…asant-to-inspect

Make test output more pleasant to inspect
* removing wrong error output

* fix host-check output

* refactor echo

---------

Co-authored-by: Chuck Pathanjali <[email protected]>
Just a quick fix to echo statement to clean it up.
previous pr had a typo causing issues on cluster
Some problems are being caused on cluster binary backup, need to revert until other fixes are in.
solvaholic and others added 5 commits November 14, 2023 23:43
Remove "contact support" from warning about route verification
Create sync file on HA replica from primary node [cluster]
…aholic/soften-warning

Backport 726 for 3.11: Remove "contact support" from warning about route verification
Backport 725 for 3.11: Create sync file on HA replica from primary node [cluster]
@amildahl amildahl changed the title 3.11 main Sync 3.11 into main Nov 28, 2023
@amildahl amildahl changed the title Sync 3.11 into main Sync 3.11-main into master Nov 28, 2023
@chuckp22
Copy link
Contributor

Overall the changes lgtm but I think the repo is missing the API_AUTH_TOKEN secret causing Integrations Tests to fail.
Cc @jianghao0718

Copy link
Contributor

@chuckp22 chuckp22 left a comment

Choose a reason for hiding this comment

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

LGTM

@amildahl amildahl merged commit 424223b into master Nov 30, 2023
17 of 19 checks passed
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.