Skip to content

Fix tas label#380

Merged
unmarshall merged 7 commits into
ai-dynamo:mainfrom
kangclzjc:fix_e2e_miss_label
Feb 1, 2026
Merged

Fix tas label#380
unmarshall merged 7 commits into
ai-dynamo:mainfrom
kangclzjc:fix_e2e_miss_label

Conversation

@kangclzjc

Copy link
Copy Markdown
Contributor

What type of PR is this?

/kind bug

What this PR does / why we need it:

Fix TAS randomly failure

Which issue(s) this PR fixes:

Fixes #379

Special notes for your reviewer:

Does this PR introduce a API change?


Additional documentation e.g., enhancement proposals, usage docs, etc.:


@Ronkahn21 Ronkahn21 left a comment

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.

Good catch finding the edge case bug, I added a few comments

Comment thread operator/e2e/setup/k8s_clusters.go Outdated
Comment thread operator/e2e/setup/k8s_clusters.go Outdated
Comment thread operator/e2e/setup/k8s_clusters.go Outdated
Comment thread operator/e2e/setup/k8s_clusters.go Outdated
@kangclzjc

Copy link
Copy Markdown
Contributor Author

@Ronkahn21 , @unmarshall PTAL

Comment thread operator/e2e/utils/k8s_client.go Outdated
Comment thread operator/e2e/utils/k8s_client.go Outdated
Comment thread operator/e2e/utils/k8s_client.go Outdated
Comment thread operator/e2e/setup/k8s_clusters.go Outdated
Comment thread operator/e2e/utils/k8s_client.go Outdated
@kangclzjc kangclzjc force-pushed the fix_e2e_miss_label branch 2 times, most recently from f245dbe to 70ecd84 Compare January 31, 2026 11:48
@kangclzjc kangclzjc requested a review from unmarshall February 1, 2026 04:27
kangclzjc and others added 7 commits February 1, 2026 15:02
Signed-off-by: kangclzjc <kangz@nvidia.com>
Signed-off-by: kangclzjc <kangz@nvidia.com>
Signed-off-by: kangclzjc <kangz@nvidia.com>
Signed-off-by: kangclzjc <kangz@nvidia.com>
Signed-off-by: kangclzjc <kangz@nvidia.com>
Signed-off-by: kangclzjc <kangz@nvidia.com>
* Using named return types for WaitAndGetReadyNode

Signed-off-by: Madhav Bhargava <madhav.bhargava@sap.com>
@unmarshall unmarshall merged commit 48b8135 into ai-dynamo:main Feb 1, 2026
7 checks passed
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.

TAS e2e test fai randomly

3 participants