Skip to content

chore: Add empty workflow so it can be run in a branch#131

Merged
danielpanzella merged 1 commit intomainfrom
danielpanzella/workflow-scaffold
Feb 11, 2026
Merged

chore: Add empty workflow so it can be run in a branch#131
danielpanzella merged 1 commit intomainfrom
danielpanzella/workflow-scaffold

Conversation

@danielpanzella
Copy link
Copy Markdown
Contributor

@danielpanzella danielpanzella commented Feb 11, 2026

Summary by CodeRabbit

  • Chores
    • Added GitHub Actions workflow to support internal chart publishing processes.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 11, 2026

📝 Walkthrough

Walkthrough

A new GitHub Actions workflow file is added for internal chart publishing. The workflow is manually triggered and includes a release job configured to run on ubuntu-latest with appropriate permissions for content access and token management, along with a repository checkout step.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/internal-chart-publish.yaml
Introduces a new "Internal Chart Publish" workflow triggered via workflow_dispatch with a release job on ubuntu-latest, read contents permissions, write id-token permissions, and a checkout step.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

  • dacbd

Poem

🐰 A workflow so neat, triggered by hand,
Publishing charts across the land,
Permissions set just right and tight,
Our release job takes flight tonight! 📊✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding a new GitHub Actions workflow file. It clearly communicates the purpose (creating an empty workflow scaffold) and the intent (enabling branch-level execution).
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch danielpanzella/workflow-scaffold

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@danielpanzella danielpanzella enabled auto-merge (squash) February 11, 2026 23:03
Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In @.github/workflows/internal-chart-publish.yaml:
- Around line 14-18: Update the GitHub Actions checkout step named "Checkout" to
use the latest supported version by changing the uses value from
actions/checkout@v2 to actions/checkout@v4; keep the existing with keys
(persist-credentials: false and fetch-depth: 0) unchanged to preserve
credentials and full history behavior, and run a quick workflow lint/CI to
verify there are no compatibility issues with the updated actions/checkout
version.

Comment on lines +14 to +18
- name: Checkout
uses: actions/checkout@v2
with:
persist-credentials: false
fetch-depth: 0
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Update actions/checkout from v2 to v4.

actions/checkout@v2 uses Node.js 12, which is no longer supported on GitHub Actions runners and will fail. Also confirmed by actionlint static analysis.

Good choices on persist-credentials: false and fetch-depth: 0.

Proposed fix
      - name: Checkout
-        uses: actions/checkout@v2
+        uses: actions/checkout@v4
        with:
          persist-credentials: false
          fetch-depth: 0
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
- name: Checkout
uses: actions/checkout@v2
with:
persist-credentials: false
fetch-depth: 0
- name: Checkout
uses: actions/checkout@v4
with:
persist-credentials: false
fetch-depth: 0
🧰 Tools
🪛 actionlint (1.7.10)

[error] 15-15: the runner of "actions/checkout@v2" action is too old to run on GitHub Actions. update the action's version to fix this issue

(action)

🤖 Prompt for AI Agents
In @.github/workflows/internal-chart-publish.yaml around lines 14 - 18, Update
the GitHub Actions checkout step named "Checkout" to use the latest supported
version by changing the uses value from actions/checkout@v2 to
actions/checkout@v4; keep the existing with keys (persist-credentials: false and
fetch-depth: 0) unchanged to preserve credentials and full history behavior, and
run a quick workflow lint/CI to verify there are no compatibility issues with
the updated actions/checkout version.

@danielpanzella danielpanzella merged commit b272e14 into main Feb 11, 2026
10 checks passed
@danielpanzella danielpanzella deleted the danielpanzella/workflow-scaffold branch February 11, 2026 23:05
j7m4 added a commit that referenced this pull request Mar 24, 2026
* chore: Upgrade go to 1.25.3 (#104)

* chore: Upgrade go to 1.25.3

* chore: Upgrade go to 1.25.3

* chore: Upgrade helm to 3.19.2 (#105)

* fix: Upgrade helm to 3.19.2 (#106)

* chore(release): version 1.21.3 [skip ci]

### [1.21.3](v1.21.2...v1.21.3) (2025-12-04)

### Bug Fixes

* Upgrade helm to 3.19.2 ([#106](#106)) ([aa4ca21](aa4ca21))

* chore: Need to create the workflow in main so it can be updated and run in v2 branch (#127)

* chore: Need to create the workflow in main so it can be updated and run in v2 branch

* chore: Add newline

* chore: Add empty workflow so it can be run in a branch (#131)

* fix to clickhouse enum we vendored (but is unused)

* work in progress on retention

* work in progress

* ONPREM-111 rentention/purge works for wandb delete

* ONPREM-111 more consistent label handling for retention

* ONPREM-111 more consistent label handling for retention

* initial retention e2e tests

* WIP: fix Redis PVC labeling for retention e2e tests

- Remove ensurePodLabels (operator SA lacks RBAC to patch pods, caused rapid requeue loop)
- Fix Redis PVC prefix: use crName-crName- pattern matching actual opstree PVC names
- Bypass cache for PVC reads in client (PVCs not watched, caused ErrCacheNotStarted)
- Add USE_EXISTING_CLUSTER=true to test-e2e-retention Makefile target
- Improve retention_test.go: namespace isolation, verifyComponentResourcesExist checks

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

* ONPREM-111 complete impl of retention DETACH/PURGE policies

---------

Co-authored-by: Kevin Chen <140976422+wandb-kc@users.noreply.github.com>
Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net>
Co-authored-by: Daniel Panzella <daniel.panzella@wandb.com>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.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.

2 participants