Lokasi ngalangkungan proxy:   [ UP ]  
[Ngawartoskeun bug]   [Panyetelan cookie]                
Skip to content

Update sql-agent-extension-automatic-registration-all-vms.md#7717

Merged
Court72 merged 1 commit intoMicrosoftDocs:livefrom
jasonplante1:patch-4
Jun 28, 2022
Merged

Update sql-agent-extension-automatic-registration-all-vms.md#7717
Court72 merged 1 commit intoMicrosoftDocs:livefrom
jasonplante1:patch-4

Conversation

@jasonplante1
Copy link
Copy Markdown
Contributor

Discussing with Abdullah Mamun on this subject - the job doesn't run daily. It runs monthly.

With that in mind, the documentation should reflect this, and the line below:

Automatic registration can take up to 4 days to detect newly created SQL Server VMs.

I doubt we can say that it will only take 4 days to pick up a VM with SQL Server installed. The process could take up to 30 depending on when SQL is installed, and when the job runs.

Currently working a case now where this is the focus of the customer's issue, and again - if the job doesn't run daily, we can't have this out in the ether.

Discussing with Abdullah Mamun on this subject - the job doesn't run daily. It runs monthly. 

With that in mind, the documentation should reflect this, and the line below:

Automatic registration can take up to 4 days to detect newly created SQL Server VMs.

I doubt we can say that it will only take 4 days to pick up a VM with SQL Server installed. The process could take up to 30 depending on when SQL is installed, and when the job runs. 

Currently working a case now where this is the focus of the customer's issue, and again - if the job doesn't run daily, we can't have this out in the ether.
@PRMerger16
Copy link
Copy Markdown
Contributor

@jasonplante1 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Court72
Copy link
Copy Markdown
Contributor

Court72 commented Jun 22, 2022

@adbadram

Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@PRMerger13 PRMerger13 added the aq-pr-triaged tracking label for the PR review team label Jun 22, 2022
@adbadram
Copy link
Copy Markdown
Contributor

We run an Automated job daily to find VM candidates where SQL Server is present, if we don't see SQL Server we will come back and revisit this VM in a month/30 days. Hence daily in this context stands correct.

@jasonplante1
Copy link
Copy Markdown
Contributor Author

jasonplante1 commented Jun 22, 2022 via email

@jasonplante1
Copy link
Copy Markdown
Contributor Author

jasonplante1 commented Jun 22, 2022 via email

@MashaMSFT
Copy link
Copy Markdown
Contributor

#sign-off

@PRMerger12
Copy link
Copy Markdown
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @adbadram

@MashaMSFT
Copy link
Copy Markdown
Contributor

@ktoliver fyi #sign-off

@PRMerger17
Copy link
Copy Markdown
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @adbadram

@Court72 Court72 merged commit 1124ff2 into MicrosoftDocs:live Jun 28, 2022
@opbld30
Copy link
Copy Markdown

opbld30 commented Jun 28, 2022

Docs Build status updates of commit e14d9c5:

✅ Validation status: passed

File Status Preview URL Details
azure-sql/virtual-machines/windows/sql-agent-extension-automatic-registration-all-vms.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

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.

9 participants