Skip to content

Remove placeholder cockpit secret gate#785

Merged
blove merged 2 commits into
mainfrom
blove/remove-cockpit-secret-placeholder
Jul 8, 2026
Merged

Remove placeholder cockpit secret gate#785
blove merged 2 commits into
mainfrom
blove/remove-cockpit-secret-placeholder

Conversation

@blove

@blove blove commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the placeholder Cockpit secret-gated integration job and CI scope key
  • drop the unused deployment-runtime integration target that only checked for COCKPIT_SECRET_TOKEN
  • mark deployment-runtime integration metadata as none until there is real non-provider CI coverage

Validation

  • node --test scripts/ci-scope.spec.mjs scripts/cockpit-matrix.spec.mjs scripts/cockpit-ports.spec.mjs scripts/ci-workflow.spec.mjs
  • NX_DAEMON=false node node_modules/nx/bin/nx.js test cockpit-registry
  • git diff --check

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview, Comment Jul 8, 2026 9:01pm

Request Review

@github-actions github-actions Bot 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.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove blove force-pushed the blove/remove-cockpit-secret-placeholder branch from 51cccd9 to 2a68e95 Compare July 8, 2026 15:21
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@blove blove merged commit 82978b7 into main Jul 8, 2026
61 checks passed
@blove blove deleted the blove/remove-cockpit-secret-placeholder branch July 8, 2026 21:32
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.

1 participant