Skip to content

feat(operator): report exact image identity (ALIEN-826) - #634

Merged
lilienblum merged 26 commits into
mainfrom
lilienblum/alien-826-operator-image-reporting
Sep 20, 2026
Merged

lilienblum merged 26 commits into
mainfrom
lilienblum/alien-826-operator-image-reporting

Conversation

@lilienblum

Copy link
Copy Markdown
Contributor

Summary

  • define and validate a backward-compatible immutable Operator image receipt
  • inject package/configured image identity into raw and product Helm manifests and report it on sync
  • forward the receipt through Manager reconcile and refresh generated core, Manager, and Platform SDK artifacts
  • compose the final ALIEN-823 lifecycle fixes so pinned consumers receive the reviewed CRD and Terraform behavior

Generated artifacts

  • pnpm run generate:manager-rust-sdk
  • SPEAKEASY_BIN at pinned 1.680.11, pnpm run generate:platform-api

Validation

  • cargo test -p alien-helm --locked: 56 passed, 1 Kind test ignored for CI
  • cargo test -p alien-manager --lib --locked: 238 passed
  • cargo test -p alien-operator --lib --locked: 61 passed
  • cargo check -p alien-platform-api --locked
  • Manager and Platform TypeScript SDK builds passed

Stacked on ALIEN-823 / #633.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 16, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-09-17T14:10:05.155479Z 3861df3 Manual request
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@greptile-apps

greptile-apps Bot commented Sep 16, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 4/5

The runtime image-reporting path appears safe, but the unrelated Platform SDK regeneration should be cleaned up before publishing these artifacts.

Fix All in CodexFindings

  1. P2 Unrelated SDK exports renamed ▶
Fix with agent prompt
### Issue 1
client-sdks/platform/typescript/src/models/operations/applycontainerregistrymanagersnapshot.ts:28
This regeneration removes existing public exports such as `Verification`, `Scope`, and `Operation` in favor of newly qualified names. These modules are re-exported through `src/models/operations/index.ts`, while the package version remains 3.3.22, so consumers importing the old symbols will fail to compile. This unrelated generated drift should be restored and released separately.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

This PR introduces an immutable Operator image receipt and carries it from generated manifests through Operator sync and Manager reconcile, while refreshing API and SDK artifacts.

  • Adds validated package/configured image identity types.
  • Injects image receipt environment variables into raw and product Helm manifests.
  • Reports the receipt during Operator sync and forwards it through Manager reconciliation.
  • Regenerates Manager and Platform API clients; the Platform regeneration also includes unrelated public-model churn that should be separated.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Installer or package resolver] -->|image and source identity| B[Helm manifest generator]
  B -->|ALIEN_OPERATOR_IMAGE_*| C[Operator configuration]
  C -->|operatorImage in sync request| D[Manager sync route]
  D -->|validated ReconcileData| E[Platform embedder]
Loading

Reviews (1) · Last reviewed commit: "test(helm): compose image reporting life..."

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: dbe209342a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: c2a173d936

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread crates/alien-operator/src/cli.rs Outdated
@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review exact commit 09c98d5. Please re-review the complete restacked diff, including the live Helm lifecycle fix inherited from #633, for correctness, security, compatibility, and missing-test blockers.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Chef's kiss.

Reviewed commit: 09c98d5371

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. You're on a roll.

Reviewed commit: 6491d3ab44

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review Exact head b91348f merges the tested ALIEN-823 rollback/uninstall lifecycle fixes beneath the operator image reporting work. The full alien-helm suite and disposable-Kind Helm/Terraform lifecycle pass on this combined head. Please review the combined exact head for correctness, compatibility, security, and lifecycle regressions.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b91348f637

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review Exact head 98cbd23 includes the latest lifecycle stack plus regenerated package dependency SDK artifacts. Please review the full exact-head diff for correctness, generated-artifact consistency, operator image reporting, lifecycle safety, compatibility, security, and missing tests.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 98cbd236f3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread client-sdks/manager/openapi.json
@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 6843fa033e

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread client-sdks/manager/openapi-3.0.json Outdated
@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. You're on a roll.

Reviewed commit: dd5aa5cf94

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review exact head . Please report only concrete correctness, security, compatibility, or verification blockers; explicitly say when none remain.

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Keep them coming!

Reviewed commit: 7c83717e60

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review

Please review the exact current head 472353f. Report only actionable correctness, security, compatibility, or verification blockers.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 472353f944

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread client-sdks/manager/openapi-3.0.json Outdated
@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Delightful!

Reviewed commit: 33855441be

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 👍

Reviewed commit: 4158862bee

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: 6c4e5b2246

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit: 6c4e5b2246

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

Codex review requested for exact rebased head bbfcbd8, based on ALIEN-823 head f6d549e. Please review the full PR, especially exact package/configured operator image identity semantics, the dedicated receipt env var, generated API/client artifacts, and compatibility for older installations. Report only actionable correctness, security, compatibility, scope, or test-evidence findings; explicitly state if none.

@lilienblum
lilienblum force-pushed the lilienblum/alien-826-operator-image-reporting branch from ef3a0c9 to bbfcbd8 Compare September 17, 2026 13:58
@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review bbfcbd8

Report expected and observed operator image receipts without colliding with deployment image overrides, and keep regenerated client artifacts scoped to the new contract.
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 🚀

Reviewed commit: bbfcbd8276

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum

Copy link
Copy Markdown
Contributor Author

@codex review 3861df3

@lilienblum
lilienblum force-pushed the lilienblum/alien-826-operator-image-reporting branch from bbfcbd8 to 3861df3 Compare September 17, 2026 14:04
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Breezy!

Reviewed commit: 3861df3e2d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@lilienblum
lilienblum added this pull request to stack #676 September 20, 2026 13:57
Base automatically changed from lilienblum/alien-823-product-helm-release-v2 to main September 20, 2026 20:03
@lilienblum
lilienblum merged commit ad81ae1 into main Sep 20, 2026
22 checks passed
@lilienblum
lilienblum deleted the lilienblum/alien-826-operator-image-reporting branch September 20, 2026 23:29
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