Skip to content

Conversation

@estebancams
Copy link
Contributor

@estebancams estebancams commented Dec 22, 2022

Reason for Change:
We still don't support windows 2004 for aks and aks-e tests are failing very often. While we determine the root cause, in order to allow PRs to go in, and since we'll still run windows 1903 tests, we can disable 2004 tests as a mitigation
A new task was reported to add win 22 tests once we migrate over from aks-e (https://msazure.visualstudio.com/One/_workitems/edit/16667635)

Issue Fixed:

Requirements:

Notes:

@estebancams estebancams marked this pull request as draft December 22, 2022 20:05
@estebancams estebancams marked this pull request as ready for review December 22, 2022 21:24
@estebancams estebancams marked this pull request as draft December 22, 2022 21:24
@estebancams estebancams marked this pull request as ready for review December 23, 2022 17:09
@estebancams estebancams changed the title wip: feat: remove windows 2004 aks-e tests from pipeline remove windows 2004 aks-e tests from pipeline Dec 23, 2022
@estebancams estebancams force-pushed the estebanca/remove-windows-2004-stage-from-pipeline branch from 645ae36 to 243f805 Compare December 23, 2022 17:20
Copy link
Contributor

@pjohnst5 pjohnst5 left a comment

Choose a reason for hiding this comment

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

LGTM as long as it's not available in AKS anyways (and aks-e is deprecated now)
It does fail quite often so in my opinion it is best to remove it while we then put the win 22 tests in for aks

Copy link
Contributor

@pjohnst5 pjohnst5 left a comment

Choose a reason for hiding this comment

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

Yeah maybe we should first get win 22 operational before removing this based on discussion offline

@estebancams estebancams closed this Jan 3, 2023
@rbtr rbtr reopened this Jan 3, 2023
@rbtr rbtr dismissed pjohnst5’s stale review January 3, 2023 18:42

dismissing per further discussion, okay to merge and previously approved

@rbtr rbtr merged commit 26dd052 into master Jan 4, 2023
@rbtr rbtr deleted the estebanca/remove-windows-2004-stage-from-pipeline branch January 4, 2023 16:32
rjdenney pushed a commit to rjdenney/azure-container-networking that referenced this pull request Jan 19, 2023
* feat: remove windos 2004 aks-e tests from pipeline

* fix: dependency on cleanup stage

Co-authored-by: Esteban Capillo <estebancams@microsoft.com>
smittal22 pushed a commit to smittal22/azure-container-networking that referenced this pull request Jan 26, 2023
* feat: remove windos 2004 aks-e tests from pipeline

* fix: dependency on cleanup stage

Co-authored-by: Esteban Capillo <estebancams@microsoft.com>
smittal22 pushed a commit to smittal22/azure-container-networking that referenced this pull request Feb 3, 2023
* feat: remove windos 2004 aks-e tests from pipeline

* fix: dependency on cleanup stage

Co-authored-by: Esteban Capillo <estebancams@microsoft.com>
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