Skip to content

Fix deploy nightly docs issue#2636

Merged
ptrendx merged 23 commits intoNVIDIA:mainfrom
pggPL:fix_github_workflows
Mar 10, 2026
Merged

Fix deploy nightly docs issue#2636
ptrendx merged 23 commits intoNVIDIA:mainfrom
pggPL:fix_github_workflows

Conversation

@pggPL
Copy link
Copy Markdown
Collaborator

@pggPL pggPL commented Jan 30, 2026

Description

This PR fixes following issues:

  • Deploy nightly docs fails, because of non-compatible packages. I tested it in my own fork and version changes fix the issue,

Fixes # (issue)

Type of change

  • Documentation change (change only to the documentation, either a fix or a new content)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Infra/Build change
  • Code refactoring

Checklist:

  • I have read and followed the contributing guidelines
  • The functionality is complete
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
@pggPL pggPL changed the title PR to debug github workflows fails PR to debug github workflows failures Jan 30, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Jan 30, 2026

Greptile Summary

This PR fixes the nightly documentation deployment workflow by upgrading GitHub Actions to compatible versions and removing deprecated parameters.

Changes

  • Upgraded actions/upload-pages-artifact from v1.0.7 to v3
  • Upgraded actions/deploy-pages from v2.0.0 to v4
  • Removed deprecated name parameter from upload-pages-artifact (v3 uses fixed name github-pages by default)
  • Added id: deployment to the deploy step for proper URL output reference in the environment configuration
  • Added workflow_dispatch trigger to allow manual workflow runs for testing

Impact

The updates ensure compatibility with current GitHub Pages Actions API and resolve the package compatibility issues mentioned in the PR description.

Confidence Score: 5/5

  • This PR is safe to merge with no risk - it only updates GitHub Actions versions and removes deprecated parameters
  • The changes are straightforward infrastructure updates that follow GitHub Actions migration guidelines. The upgrade from v1/v2 to v3/v4 is necessary for compatibility, and the removal of the deprecated name parameter is required by v3. The addition of workflow_dispatch and id: deployment are best practices. No code logic is changed, only workflow configuration.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/deploy_nightly_docs.yml Updates GitHub Actions to v3/v4 for compatibility, removes deprecated parameters, and adds manual trigger support

Last reviewed commit: 8ac0406

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/deploy_nightly_docs.yml
@pggPL pggPL changed the title PR to debug github workflows failures Fix Github workflows issues Jan 30, 2026
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
pggPL added 2 commits February 3, 2026 04:55
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, 2 comments

Edit Code Review Agent Settings | Greptile

Comment thread .github/workflows/build.yml Outdated
Comment thread .github/workflows/build.yml Outdated
Restore .github/workflows/build.yml to upstream/main state.
Only deploy_nightly_docs.yml changes are relevant to this PR.

Signed-off-by: Pawel Gadzinski <pgadzinski@nvidia.com>
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@pggPL pggPL force-pushed the fix_github_workflows branch from ff94345 to 8ac0406 Compare February 24, 2026 12:24
Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@pggPL pggPL changed the title Fix Github workflows issues Fix deploy nightly docs issue Feb 24, 2026
@ptrendx ptrendx merged commit 3846bf7 into NVIDIA:main Mar 10, 2026
10 of 12 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.

2 participants