generate-registry-metadata: generate metadata individually for steps#1222
Conversation
This PR changes the way that metadata gets generated so that each step has its own metadata file that gets stored in the same directory as the step. This allows PRs that update metadata (like OWNERS file updates) to be approved by the approvers of the individual steps instead of requiring an approval by the dptp team as was the case with the monolithic metadata file. This also removes the separate `registry-metadata` directory and adds the OWNERS and metadata files to the main `registry` directory. This was done to make sure that the `configresolver` can load a registry with metadata, which was not the case when a separate registry directory was being used. A new make target has been added to verify that the metadata is up-to-date and be added as a unit test to CI.
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: AlexNPavel, stevekuznetsov The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
/retest Please review the full test history for this PR and help us cut down flakes. |
|
@AlexNPavel: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
This PR changes the way that metadata gets generated so that each step
has its own metadata file that gets stored in the same directory as the
step. This allows PRs that update metadata (like OWNERS file updates) to
be approved by the approvers of the individual steps instead of
requiring an approval by the dptp team as was the case with the
monolithic metadata file.
This also removes the separate
registry-metadatadirectory and addsthe OWNERS and metadata files to the main
registrydirectory. This wasdone to make sure that the
configresolvercan load a registry withmetadata, which was not the case when a separate registry directory was
being used. A new make target has been added to verify that the metadata
is up-to-date and be added as a unit test to CI.