Skip to content
This repository has been archived by the owner on Jun 28, 2023. It is now read-only.

Create OpenAPIV3 schema for metrics-server package and generate package with the schema #2758

Merged
merged 1 commit into from
Jan 13, 2022

Conversation

blc1996
Copy link
Contributor

@blc1996 blc1996 commented Jan 4, 2022

Signed-off-by: Lucheng Bao luchengb@vmware.com

What this PR does / why we need it

Create OpenAPIV3 schema for metrics-server package and generate package with the schema

Details for the Release Notes (PLEASE PROVIDE)

Create OpenAPIV3 schema for metrics-server package and generate package with the schema

Which issue(s) this PR fixes

Fixes: #2803

Describe testing done for PR

Tested with ytt commands

$ make generate-openapischema-package PACKAGE=metrics-server VERSION=0.5.1

Special notes for your reviewer

YTT version needs to be bumped to latest (0.38.0)

@blc1996 blc1996 added the owner/packages Work executed by a package's maintainer label Jan 4, 2022
@blc1996 blc1996 requested a review from a team as a code owner January 4, 2022 07:58
@blc1996 blc1996 changed the title Create OpenAPIV3 schema for metrics-server package and generate package with the schema [WIP] Create OpenAPIV3 schema for metrics-server package and generate package with the schema Jan 4, 2022
@github-actions github-actions bot added the owner/docs Work executed by VMware documentation team label Jan 4, 2022
@shivaani0505
Copy link
Contributor

@blc1996 you need to create a separate issue in tce and fix "Which issue(s) this PR fixes" section, Issue mentioned there is for Secret Gen controller

@blc1996 blc1996 force-pushed the luchengb/openapi-metrics-server branch from d25b4cb to 179ad00 Compare January 11, 2022 00:44
@blc1996 blc1996 changed the title [WIP] Create OpenAPIV3 schema for metrics-server package and generate package with the schema Create OpenAPIV3 schema for metrics-server package and generate package with the schema Jan 11, 2022
Signed-off-by: Lucheng Bao <luchengb@vmware.com>
@blc1996 blc1996 force-pushed the luchengb/openapi-metrics-server branch from 179ad00 to b6f28a8 Compare January 12, 2022 00:53
@blc1996
Copy link
Contributor Author

blc1996 commented Jan 13, 2022

Hi @ggpaue , could you take a look at this MR?

@ggpaue
Copy link
Contributor

ggpaue commented Jan 13, 2022

@blc1996 the change looks good to me and I will approve it, can you add what ytt commands you used for testing in the description before merge? thanks

Copy link
Contributor

@ggpaue ggpaue left a comment

Choose a reason for hiding this comment

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

lgtm

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required owner/docs Work executed by VMware documentation team owner/packages Work executed by a package's maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create OpenAPIV3 schema for metrics-server in TCE
4 participants