Skip to content

ci: AUT-2179 temporarily disable Fireworks E2E test - #2871

Merged
kajalj22 merged 1 commit into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:ci/temporarily-disable-fireworks-e2e
Aug 30, 2026
Merged

kajalj22 merged 1 commit into
NVIDIA-NeMo:mainfrom
svcnemo-autobot:ci/temporarily-disable-fireworks-e2e

Conversation

@svcnemo-autobot

Copy link
Copy Markdown
Contributor

Background

Nightly run 33223378543 failed when the live Fireworks smoke test produced only an agent_run_error rollout.

What changed

  • Temporarily disable the Fireworks provider E2E job with an explicit workflow comment.
  • Allow its expected skipped result in the nightly summary while continuing to reject failures.
  • Cover the temporary condition and summary behavior in CI tests.

Details

flowchart LR
  A[Nightly CI] --> B[Fireworks E2E skipped]
  B --> C[Summary accepts skip]
  C --> D[Other nightly gates remain enforced]
Loading

Tested

  • tests/unit_tests/test_ci_environment.py: 69 passed.
  • Workflow YAML parsed and temporary skip semantics verified.
  • Full pre-commit/core gates were attempted but blocked by the typed checkout lacking local Git metadata and a Git executable; GitHub CI remains authoritative.

Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
@svcnemo-autobot
svcnemo-autobot requested a review from a team as a code owner August 29, 2026 10:30
@copy-pr-bot

copy-pr-bot Bot commented Aug 29, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@kajalj22

Copy link
Copy Markdown
Contributor

/ok to test 1a88045

@kajalj22
kajalj22 enabled auto-merge (squash) August 30, 2026 01:48
@kajalj22
kajalj22 merged commit 40b1001 into NVIDIA-NeMo:main Aug 30, 2026
39 checks passed
jkyi-nvidia pushed a commit that referenced this pull request Sep 11, 2026
## Background

Nightly run 33223378543 failed when the live Fireworks smoke test
produced only an `agent_run_error` rollout.

## What changed

- Temporarily disable the Fireworks provider E2E job with an explicit
workflow comment.
- Allow its expected skipped result in the nightly summary while
continuing to reject failures.
- Cover the temporary condition and summary behavior in CI tests.

## Details

```mermaid
flowchart LR
  A[Nightly CI] --> B[Fireworks E2E skipped]
  B --> C[Summary accepts skip]
  C --> D[Other nightly gates remain enforced]
```

## Tested

- `tests/unit_tests/test_ci_environment.py`: 69 passed.
- Workflow YAML parsed and temporary skip semantics verified.
- Full pre-commit/core gates were attempted but blocked by the typed
checkout lacking local Git metadata and a Git executable; GitHub CI
remains authoritative.

Signed-off-by: svcnemo-autobot <svcnemo-autobot@nvidia.com>
Signed-off-by: Junkeun Yi <jkyi@nvidia.com>

This branch was previously deployed

1 inactive deployment
public β€” 1a880458 Deployed Aug 30, 2026 by copy-pr-bot[bot] via release / finalize / notify #2300
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.

2 participants