Skip to content

[FluidRelay] Generate fluid-relay with CodeGen v2#5155

Merged
necusjz merged 13 commits into
Azure:mainfrom
ZengTaoxu:fluidrelay
Jul 27, 2022
Merged

[FluidRelay] Generate fluid-relay with CodeGen v2#5155
necusjz merged 13 commits into
Azure:mainfrom
ZengTaoxu:fluidrelay

Conversation

@ZengTaoxu
Copy link
Copy Markdown
Contributor

@ZengTaoxu ZengTaoxu commented Jul 26, 2022


Closes: Azure/azure-cli#22938

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

Related command

az fluid-relay server create
az fluid-relay server update
az fluid-relay server list
az fluid-relay server list-key
az fluid-relay server regenerate-key
az fluid-relay server show
az fluid-relay server delete
az fluid-relay container show
az fluid-relay container list
az fluid-relay container delete

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?

For new extensions:

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.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

@ghost ghost added the Auto-Assign Auto assign by bot label Jul 26, 2022
@ghost ghost requested review from kairu-ms and necusjz July 26, 2022 05:47
@ghost ghost assigned kairu-ms Jul 26, 2022
@ghost ghost added this to the Jul 2022 (2022-08-02) milestone Jul 26, 2022
@ghost ghost requested a review from yonzhan July 26, 2022 05:47
@ghost ghost added the Code Gen label Jul 26, 2022
@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Jul 26, 2022

FluidRelay

Comment thread src/fluid-relay/README.md Outdated
This is an extension to Azure CLI to manage FluidRelay resources.

## How to use ##
Please add commands usage here. No newline at end of file
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's better to have some basic use cases.

confirmation="Are you sure you want to perform this operation?",
)
class Delete(AAZCommand):
"""Delete a Fluid Relay container.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's better to complete the examples part, same as other commands.

Comment thread src/service_name.json Outdated
{
"Command": "az fluid-relay",
"AzureServiceName": "Azure Fluid Relay",
"URL": "https://docs.microsoft.com/en-us/azure/azure-fluid-relay/overview"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Page not found?

@@ -0,0 +1,4 @@
{
"azext.isExperimental": true,
"azext.minCliCoreVersion": "2.38.0"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should wait for the release of Azure CLI 2.39.0

Suggested change
"azext.minCliCoreVersion": "2.38.0"
"azext.minCliCoreVersion": "2.39.0"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you for your correction! Relevant questions have been revised.

@necusjz necusjz changed the title [FluidRelay]Generate fluid-relay with codegen v2 [FluidRelay] Generate fluid-relay with CodeGen v2 Jul 26, 2022
Comment thread src/fluid-relay/README.md Outdated
This is an extension to Azure CLI to manage FluidRelay resources.

## How to use ##
Please add commands usage here.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We can remove this hint line. :)

@necusjz necusjz merged commit 1dab6ae into Azure:main Jul 27, 2022
necusjz added a commit that referenced this pull request Jul 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Auto-Assign Auto assign by bot Code Gen

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Terraform and CLI work for GA of Azure Fluid Relay

4 participants