Skip to content

Clarify that env-vars is ignored unless image is specified for contai… - #34079

Open
Alabi Toluwaleke (fvlga) wants to merge 1 commit into
Azure:devfrom
fvlga:fix-containerapp-job-env-vars-doc
Open

Alabi Toluwaleke (fvlga) wants to merge 1 commit into
Azure:devfrom
fvlga:fix-containerapp-job-env-vars-doc

Conversation

@fvlga

Copy link
Copy Markdown

Related command

az containerapp job create

Description

Addressing #32574.

The --env-vars parameter on az containerapp job create is silently
ignored unless --image is also provided. This adds a note to the
--env-vars help text specifically for containerapp job create
(via a command-scoped argument_context override) clarifying this
behavior, without affecting the --env-vars help text used by other
containerapp commands.

Testing Guide

Ran az containerapp job create --help and confirmed the --env-vars
help text now includes the note about it being ignored unless
--image is specified. No functional/behavioral change — help text
only.


This checklist is used to make sure that common guidelines for a pull request are followed.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI 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.

🟢 Approval recommended

Help text-only change with no unresolved issues.

Pull request overview

Clarifies that --env-vars is ignored for az containerapp job create unless --image is specified.

Changes:

  • Adds command-scoped help text for this behavior.
File summaries
File Description
src/azure-cli/azure/cli/command_modules/containerapp/_params.py Updates job-create environment variable help text.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@yonzhan

Copy link
Copy Markdown
Collaborator

containerapp

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.

5 participants