Skip to content
This repository has been archived by the owner on Feb 14, 2024. It is now read-only.

ci: increase the default apply timeout #353

Merged
merged 3 commits into from
Oct 27, 2023
Merged

ci: increase the default apply timeout #353

merged 3 commits into from
Oct 27, 2023

Conversation

FoSix
Copy link
Contributor

@FoSix FoSix commented Oct 26, 2023

Description

Increase apply_timeout for infrastructure building workflows.

Motivation and Context

Since our code can sometimes be quite complex, default 30 minutes might not be enough to perform apply or idempotence tests. This gives an option to override this property from the calling workflow level.

Additionally, chatops command had no possibility to override this parameter. This change adds that possibility.

How Has This Been Tested?

tested on Azure repo copy

Screenshots (if appropriate)

Types of changes

  • CI

Checklist

  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes if appropriate.
  • All new and existing tests passed.

@FoSix FoSix requested a review from a team as a code owner October 26, 2023 14:48
@FoSix FoSix force-pushed the apply_timeout-increase branch from 3d5ded7 to d947ac6 Compare October 26, 2023 14:53
Copy link
Contributor

@pimielowski pimielowski left a comment

Choose a reason for hiding this comment

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

Lgtm

@FoSix FoSix merged commit 4914c10 into main Oct 27, 2023
1 check passed
@FoSix FoSix deleted the apply_timeout-increase branch October 27, 2023 07:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants