Skip to content

Add a new annotation to define a env-file for the run command per container#73

Open
fabiosammy wants to merge 1 commit into
1Password:mainfrom
fabiosammy:patch-1
Open

Add a new annotation to define a env-file for the run command per container#73
fabiosammy wants to merge 1 commit into
1Password:mainfrom
fabiosammy:patch-1

Conversation

@fabiosammy

@fabiosammy fabiosammy commented Apr 30, 2026

Copy link
Copy Markdown

✨ Summary

Added support for run environment file annotations in container mutation.

🔗 Resolves:

So, the idea behind this is to use a file that is already in the container, defining the list of environments instead of updating the k8s manifests every time a new environment is needed.

✅ Checklist

  • 🖊️ Commits are signed
  • 🧪 Tests added/updated: (See the Testing Guide for when to use each type and how to run them)
    • 🔹 Unit
    • 🔸 Integration
    • 🌐 E2E (Connect)
    • 🔑 E2E (Service Account)
  • 📚 Docs updated (if behavior changed)

🕵️ Review Notes & ⚠️ Risks

This is my first time contributing to a "go" repository. I did it using the Claude AI, so sorry about that.
I feel that this should not be here, but a new environment should be available, something like OP_RUN_ENV_FILE, but I could not figure out which project I should contribute to.

Added support for run environment file annotations in container mutation.
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