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

Fix wrong backup check for SQL Always on High availability Groups #518

Closed
wants to merge 0 commits into from

Conversation

marcohald
Copy link
Contributor

Bug reports

operating system name and version
SQL Always on High availability Groups on SQL Server 2019
Windows Server 2019
Detailed steps to reproduce the bug

  1. Create a Backup of a High Available Database on Server1
  2. Move the Database to Server2
  3. You get a backup Error because no actual backup is found

Proposed changes

What is the expected behavior?
The Agent should return the Backup status, even if it is not the primary Server
What is the observed behavior?
The Agent does not return the Backup if it is not the Primary Replica
If it's not obvious from the above: In what way does your patch change the current behavior?
The Primary Server check is removed
Is this a new problem? What made you submit this PR (new firmware, new device, changed device behavior)?
The move of the datbase.

@TimotheusBachinger
Copy link
Contributor

Dear Checkmk Contributor! Unfortunately, we had to re-write our git-repo history, rendering your PR auto-closed. We will therefore rebase your PR onto the current master and reopen it again. Sorry for the inconvenience.

@TimotheusBachinger
Copy link
Contributor

Dear Contributor. Unfortunately, we learned that re-opening a PR which was force-rebased, is not possible (see isaacs/github#361). Therefore we kindly ask you to create a new PR for your change. We apologize for the circumstances and will implement technical measures to prevent such incidents in the future.

@marcohald
Copy link
Contributor Author

recreated at #727

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants