Add ACA dynamic session NodeLTS support in Azure CLI

[ASHIQUEMD]

This PR includes below changes.

• Adding new type NodeLTS for container_type parameter in az containerapp sessionpool create command.

Related command

Create Session Pool

az containerapp sessionpool create \
    --name <session-pool-name> \
    --resource-group <RESOURCE_GROUP> \
    --location westus2 \
    --container-type NodeLTS \
    --max-sessions 100 \
    --cooldown-period 300 \
    --network-status EgressDisabled

Execute Code

az containerapp session code-interpreter execute -n <session-pool-name> -g <RESOURCE_GROUP> --identifier <identifier > --code "console.log(\"inline-synchronous\")" --timeout-in-seconds 30 --session-pool-location northcentralusstage

Scenarios tested

1. Creating session pool with NodeLTS as container-type
2. Executed nodejs code using Azure cli command.

General Guidelines

• Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
• Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
• My extension version conforms to the Extension version schema

For new extensions:

• My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

[azure-client-tools-bot-prd]

⚠️Azure CLI Extensions Breaking Change Test

⚠️containerapp

rule	cmd_name	rule_message	suggest_message
⚠️ 1010 - ParaPropUpdate	containerapp sessionpool create	cmd containerapp sessionpool create update parameter container_type: updated property choices from ['CustomContainer', 'PythonLTS'] to ['CustomContainer', 'NodeLTS', 'PythonLTS']

[azure-client-tools-bot-prd]

Hi @ASHIQUEMD,
Since the current milestone time is less than 7 days, this pr will be reviewed in the next milestone.

[azure-client-tools-bot-prd]

Hi @ASHIQUEMD,
Please write the description of changes which can be perceived by customers into HISTORY.rst.
If you want to release a new extension version, please update the version in setup.py as well.

[yonzhan]

Thank you for your contribution! We will review the pull request and get back to you soon.

[github-actions]

CodeGen Tools Feedback Collection

Thank you for using our CodeGen tool. We value your feedback, and we would like to know how we can improve our product. Please take a few minutes to fill our codegen survey

[ASHIQUEMD]

Hi @ASHIQUEMD Can you show me where is the code for in terms of code execution, the container type will be derived from the session pool name, so I believe no changes are needed in that area? I check the code but didn't find it. Thanks.

And please add a test for command with argument and new value when the sessionpool name not contains container type: --container-type NodeLTS az containerapp sessionpool create -g {resource_group} -n {sessionpool_name_custom} --container-type NodeLTS

Here is the test

[Greedygre]

About the test failed:
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_env_commands.py::ContainerappEnvScenarioTest::test_containerapp_env_infrastructure_rg
FAILED src/containerapp/azext_containerapp/tests/latest/test_containerapp_nfsazurefile.py::ContainerAppMountNfsAzureFileTest::test_container_app_mount_nfsazurefile_e2e

I will rerun and update the recording files in PR: #7913

[yanzhudd]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[Juliehzl]

why do you need to create env?

[Greedygre]

Hi @ASHIQUEMD
The env_name is unused, please remove it is not necessary:
The code about creating env at test_containerapp_session_code_interpreter also not needed

azure-cli-extensions/src/containerapp/azext_containerapp/tests/latest/test_containerapp_session_code_interpreter.py

Line 24 in f0677b5

env_name = self.create_random_name(prefix='aca-sp-env', length=24)

[yanzhudd]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).

[yanzhudd]

please fix the CI issues

[Greedygre]

please fix the CI issues

Hi @yanzhudd
I update the recording files yesterday due to network update their api-version, but they revert their change and break our CI:
Azure/azure-cli@90f7650

What should we do? Thanks

[yanzhudd]

please fix the CI issues

Hi @yanzhudd I update the recording files yesterday due to network update their api-version, but they revert their change and break our CI: Azure/azure-cli@90f7650

What should we do? Thanks

Hi @Greedygre sorry for the inconvenience. Please revert the PR that updated the API version as well.

[Greedygre]

After this PR:#7919 merged, sync main branch the CI will fix.

[Greedygre]

Hi @ASHIQUEMD
Please sync main branch to fix the CI issue.