Skip to content

pkg/metadata: bundle metadata utils and types#256

Closed
bcrochet wants to merge 3 commits intooperator-framework:masterfrom
bcrochet:feature/find-metadata
Closed

pkg/metadata: bundle metadata utils and types#256
bcrochet wants to merge 3 commits intooperator-framework:masterfrom
bcrochet:feature/find-metadata

Conversation

@bcrochet
Copy link
Copy Markdown
Contributor

From the original PR:

There should be a canonical set of utilities and types that can find and parse an annotations.yaml in a bundle directory instead of re-implementing that logic in various ways in various operator-framework repos. This repo is a good home for them because it is the source of truth for many OF APIs.

/kind feature

Rebase of #60

@openshift-ci openshift-ci Bot added the kind/feature Categorizes issue or PR as related to a new feature. label Aug 18, 2022
@openshift-ci openshift-ci Bot requested review from dinhxuanvu and njhale August 18, 2022 16:52
@bcrochet bcrochet force-pushed the feature/find-metadata branch from 966c2aa to b374b6a Compare August 19, 2022 13:02
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 17, 2022
@bcrochet bcrochet force-pushed the feature/find-metadata branch from b374b6a to c999d46 Compare September 18, 2022 17:33
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 18, 2022
@bcrochet bcrochet force-pushed the feature/find-metadata branch from c999d46 to 22599e7 Compare September 19, 2022 13:59
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 30, 2022
@bcrochet bcrochet force-pushed the feature/find-metadata branch from 22599e7 to f83328d Compare October 3, 2022 15:21
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 3, 2022
@bcrochet bcrochet force-pushed the feature/find-metadata branch from f83328d to 162dfd8 Compare November 15, 2022 12:52
@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 24, 2023
Eric Stroczynski added 3 commits February 27, 2023 08:57
and return the full AnnotationsFile from FindAnnotations()

pkg/manifests: move metadata types to pkg/metadata
@bcrochet bcrochet force-pushed the feature/find-metadata branch from 162dfd8 to f18f4a7 Compare February 27, 2023 14:00
@openshift-merge-robot openshift-merge-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 27, 2023
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Feb 27, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: bcrochet
Once this PR has been reviewed and has the lgtm label, please assign awgreene for approval. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jun 28, 2023
@openshift-merge-robot
Copy link
Copy Markdown

PR needs rebase.

Details

Instructions 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.

@perdasilva
Copy link
Copy Markdown
Contributor

closing as stale

@perdasilva perdasilva closed this Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/feature Categorizes issue or PR as related to a new feature. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants