Skip to content

Conversation

@fmenezes
Copy link
Collaborator

@fmenezes fmenezes commented Aug 18, 2022

Proposed changes

Datadog integration is disabled in cloudgov

Jira ticket: CLOUDP-132962

Closes #[issue number]

Checklist

  • I have signed the MongoDB CLA
  • I have added tests that prove my fix is effective or that my feature works
  • I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
  • I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
  • I have updated test/README.md (if an e2e test has been added)
  • I have run make fmt and formatted my code

Further comments

@fmenezes fmenezes marked this pull request as ready for review August 18, 2022 10:53
@fmenezes fmenezes requested a review from a team August 18, 2022 10:53
Copy link
Contributor

@Razeer123 Razeer123 left a comment

Choose a reason for hiding this comment

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

LGTM

@fmenezes fmenezes merged commit 784de4e into master Aug 18, 2022
@fmenezes fmenezes deleted the CLOUDP-132962 branch August 18, 2022 11:02
Comment on lines +51 to +73
if !Gov() { // cloudgov does not have an available datadog region
t.Run("Create DATADOG", func(t *testing.T) {
cmd := exec.Command(cliPath,
integrationsEntity,
"create",
datadogEntity,
"--apiKey",
key,
"--projectId",
g.projectID,
"-o=json")
cmd.Env = os.Environ()
resp, err := cmd.CombinedOutput()

a := assert.New(t)
a.NoError(err, string(resp))

var thirdPartyIntegrations mongodbatlas.ThirdPartyIntegrations
if err := json.Unmarshal(resp, &thirdPartyIntegrations); a.NoError(err) {
a.True(integrationExists(datadogEntity, thirdPartyIntegrations))
}
})
}
Copy link
Contributor

Choose a reason for hiding this comment

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

t.Skip("Skipping testing becuase of gov")

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.

4 participants