Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[extension/sigv4authextension] update to use generated status #21285

Closed

Conversation

my-git9
Copy link
Member

@my-git9 my-git9 commented May 1, 2023

Description:

This updates the metadata for the extension to include stability information used to generate the status table.

Link to tracking Issue:
#19172

Testing:

Documentation:

@my-git9 my-git9 requested a review from a team May 1, 2023 15:20
@my-git9 my-git9 requested a review from Aneurysm9 as a code owner May 1, 2023 15:20
@github-actions github-actions bot requested a review from erichsueh3 May 1, 2023 15:20
@my-git9 my-git9 force-pushed the sigv4authextensionstatus branch 3 times, most recently from 77ec6cd to 178864e Compare May 3, 2023 09:48
Signed-off-by: xin.li <xin.li@daocloud.io>
@my-git9 my-git9 force-pushed the sigv4authextensionstatus branch from 178864e to 23c98f2 Compare May 3, 2023 09:57
Copy link
Contributor

@codeboten codeboten left a comment

Choose a reason for hiding this comment

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

Please rebase on main and generate. I think I've got all the changes in the suggestions, but make generate will make sure CI doesnt fail :)


status:
class: extension
stability: beta
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
stability: beta
stability:
beta: [extension]

@@ -30,7 +32,7 @@ const (

// NewFactory creates a factory for the Sigv4 Authenticator extension.
func NewFactory() extension.Factory {
return extension.NewFactory(typeStr, createDefaultConfig, createExtension, component.StabilityLevelBeta)
return extension.NewFactory(typeStr, createDefaultConfig, createExtension, metadata.Stability)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
return extension.NewFactory(typeStr, createDefaultConfig, createExtension, metadata.Stability)
return extension.NewFactory(typeStr, createDefaultConfig, createExtension, metadata.ExtensionStability)

Comment on lines +10 to +11
Type = "sigv4authextension"
Stability = component.StabilityLevelBeta
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
Type = "sigv4authextension"
Stability = component.StabilityLevelBeta
Type = "sigv4authextension"
ExtensionStability = component.StabilityLevelBeta

@codeboten codeboten added the Skip Changelog PRs that do not require a CHANGELOG.md entry label May 5, 2023
@my-git9
Copy link
Member Author

my-git9 commented May 11, 2023

This PR conflicts with #20326, I will close it.
Thanks for all reviewers!

@my-git9 my-git9 closed this May 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension/sigv4auth Skip Changelog PRs that do not require a CHANGELOG.md entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants