Skip to content

[MA-669] Add support for /v1/downtime/cancel/by_scope.#488

Merged
armcburney merged 5 commits intomasterfrom
armcburney/downtime_cancel_by_scope_endpoint
Dec 10, 2019
Merged

[MA-669] Add support for /v1/downtime/cancel/by_scope.#488
armcburney merged 5 commits intomasterfrom
armcburney/downtime_cancel_by_scope_endpoint

Conversation

@armcburney
Copy link
Copy Markdown
Member

@armcburney
Copy link
Copy Markdown
Member Author

/azp run DataDog.datadogpy.integration

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@armcburney
Copy link
Copy Markdown
Member Author

/azp run DataDog.datadogpy.integration

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Contributor

@nmuesch nmuesch left a comment

Choose a reason for hiding this comment

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

Thanks for this! Left two minor comments


# Create downtime with scope_one
end = start + 1000
downtime_one = dog.Downtime.create(scope=scope_one, start=start, end=end)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thinking about subsequent test runs here. Would creating a downtime that already exists and is disabled just flip the disabled flag back to False?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ahh no, it will create a separate downtime with a different downtime["id"].

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Sorry, to clarify: it creates a new one because the start and end will be different depending on the test execution - so you schedule it for a different time period.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Ahh ok that sounds good to me.

@armcburney
Copy link
Copy Markdown
Member Author

/azp run DataDog.datadogpy.integration

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@armcburney armcburney requested a review from nmuesch December 10, 2019 23:01
Copy link
Copy Markdown
Contributor

@nmuesch nmuesch left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@armcburney armcburney merged commit 9da1e45 into master Dec 10, 2019
@armcburney armcburney deleted the armcburney/downtime_cancel_by_scope_endpoint branch December 10, 2019 23:08
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.

5 participants