azure-sdk-for-go icon indicating copy to clipboard operation
azure-sdk-for-go copied to clipboard

Add profiles generation tool and init version of profiles

Open tadelesh opened this issue 3 years ago • 4 comments

Partially resolve https://github.com/Azure/azure-sdk-for-go/issues/16830.

  • Add some packages to generate mgmt. plane config and SDK for profile.
  • First version of profile config 2020-09-01-hybrid package which only contains mgmt. plane modules.

tadelesh avatar Aug 10 '22 13:08 tadelesh

/azp run prepare-pipelines

tadelesh avatar Aug 10 '22 13:08 tadelesh

Azure Pipelines successfully started running 1 pipeline(s).

azure-pipelines[bot] avatar Aug 10 '22 13:08 azure-pipelines[bot]

Few questions.

  • Does this include full generated code, i.e. no removal of resource types?
  • When will data-plane parts (i.e. key vault) be added?

jhendrixMSFT avatar Aug 10 '22 16:08 jhendrixMSFT

Few questions.

  • Does this include full generated code, i.e. no removal of resource types?
  • When will data-plane parts (i.e. key vault) be added?
  • Does this include full generated code, i.e. no removal of resource types? Yes. I found it's hard to pair the config with operation/operation group. For example, which operation belong to storageAccounts/blobServices in the spec according to this config. If I could figure out how to pair them, I think I could find some way to remove useless resource types.

  • When will data-plane parts (i.e. key vault) be added? Since we will announce the deprecation for track1 publicly later this September, I think we need to finish the work before that. According to the issue, it seems we don't have a conclusion for how we add the data plane modules. Is there any update for this?

tadelesh avatar Aug 11 '22 03:08 tadelesh

I've moved tooling change out of this PR and currently this PR should be a clear beta version for profile hybrid20200901, but only support mgmt. plane.

tadelesh avatar Oct 24 '22 07:10 tadelesh

I see that the root build.go file was removed. While you don't have to fix it now, we really should ensure that all modules can be built via go generate.

jhendrixMSFT avatar Nov 02 '22 15:11 jhendrixMSFT

I see that the root build.go file was removed. While you don't have to fix it now, we really should ensure that all modules can be built via go generate.

OK. I think if we decide to use profile, I'll do the fix.

tadelesh avatar Nov 03 '22 01:11 tadelesh