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.

Sign up for GitHub

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

Jump to bottom

OADP-3055: Add more keywords, categories and change maintainers to group e-mail #1304

Merged
merged 1 commit into from Feb 14, 2024
Merged

OADP-3055: Add more keywords, categories and change maintainers to group e-mail #1304

merged 1 commit into from Feb 14, 2024

Conversation

mpryc
Copy link
Contributor

@mpryc mpryc commented Feb 1, 2024 •
edited by openshift-ci bot

Partially fixes improvements as defined in the issue OADP-3055

  • Add maintainers e-mail group instead of individuals
  • Adds keywords to be able to find the operator easier within OperatorHub
  • Used additional categories, so it's easier to find within OperatorHub:
    • Cloud Provider - integrations with various cloud providers to facilitate backup and restore operations
    • Developer Tools - used in development and operations workflows for managing backups and restores
    • Modernization & Migration - used during migration and modernization efforts when moving applications to and from OCP clusters
    • OpenShift Optional - current category, won't be changing that
    • Storage - closely associated with storage-related operations

@mpryc
OADP-3055: Add more keywords, categories and change maintainers to gr…
6bbc1bb 
…oup e-mail

Partially fixes improvements as defined in the #OADP-3055
 - Add maintainers e-mail group instead of individuals
 - Adds keywords to be able to find the operator easier within OperatorHub
 - Used additional categories, so it's easier to find within OperatorHub:
   Cloud Provider
     integrations with various cloud providers to facilitate backup and restore operations
   Developer Tools
     used in development and operations workflows for managing backups and restores
   Modernization & Migration
     used during migration and modernization efforts when moving applications to and from OCP clusters
   OpenShift Optional
     current category, won't be changing that
   Storage
     closely associated with storage-related operations

Signed-off-by: Michal Pryc <mpryc@redhat.com>
@openshift-ci-robot openshift-ci-robot added the jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. label Feb 1, 2024
@openshift-ci-robot
Copy link

openshift-ci-robot commented Feb 1, 2024 •
edited by openshift-ci bot

@mpryc: This pull request references OADP-3055 which is a valid jira issue.

Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the bug to target either version "4.16." or "openshift-4.16.", but it targets "OADP 1.3.0" instead.

In response to this:

Partially fixes improvements as defined in the issue OADP-3055

  • Add maintainers e-mail group instead of individuals
  • Adds keywords to be able to find the operator easier within OperatorHub
  • Used additional categories, so it's easier to find within OperatorHub:
    • Cloud Provider - integrations with various cloud providers to facilitate backup and restore operations
    • Developer Tools - used in development and operations workflows for managing backups and restores
    • Modernization & Migration - used during migration and modernization efforts when moving applications to and from OCP clusters
    • OpenShift Optional - current category, won't be changing that
    • Storage - closely associated with storage-related operations

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 openshift-eng/jira-lifecycle-plugin repository.

@mpryc
Copy link
Contributor Author

mpryc commented Feb 1, 2024

/test 4.14-e2e-test-aws

kaovilai
kaovilai approved these changes Feb 9, 2024
View reviewed changes
Copy link
Member

@kaovilai kaovilai left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Feb 9, 2024
@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Feb 9, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kaovilai, mpryc, weshayutin

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

The pull request process is described here

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-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 9, 2024
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 314cb26 and 2 for PR HEAD 6bbc1bb in total

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD 20ee343 and 1 for PR HEAD 6bbc1bb in total

@kaovilai
Copy link
Member

/override ci/prow/4.12-e2e-test-aws
/override ci/prow/4.13-e2e-test-aws
/override ci/prow/4.14-e2e-test-aws

no need, only csv description updates.

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Feb 12, 2024

@kaovilai: Overrode contexts on behalf of kaovilai: ci/prow/4.12-e2e-test-aws, ci/prow/4.13-e2e-test-aws, ci/prow/4.14-e2e-test-aws

In response to this:

/override ci/prow/4.12-e2e-test-aws
/override ci/prow/4.13-e2e-test-aws
/override ci/prow/4.14-e2e-test-aws

no need, only csv description updates.

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.

@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD bb2fb74 and 0 for PR HEAD 6bbc1bb in total

@openshift-ci-robot
Copy link

/hold

Revision 6bbc1bb was retested 3 times: holding

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 13, 2024
@mpryc
Copy link
Contributor Author

mpryc commented Feb 13, 2024

/retest

2 similar comments
@mpryc
Copy link
Contributor Author

mpryc commented Feb 13, 2024

/retest

@mpryc
Copy link
Contributor Author

mpryc commented Feb 13, 2024

/retest

@openshift-ci OpenShift CI
Copy link

openshift-ci bot commented Feb 13, 2024

@mpryc: all tests passed!

Full PR test history. Your PR dashboard.

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. I understand the commands that are listed here.

@weshayutin weshayutin removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Feb 14, 2024
@openshift-merge-bot openshift-merge-bot bot merged commit 1f088fe into openshift:master Feb 14, 2024
16 checks passed
@kaovilai
Copy link
Member

/cherry-pick oadp-1.3

@openshift-cherrypick-robot
Copy link
Contributor

@kaovilai: new pull request created: #1329

In response to this:

/cherry-pick oadp-1.3

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.

@openshift-cherrypick-robot openshift-cherrypick-robot mentioned this pull request Feb 14, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@weshayutin weshayutin weshayutin approved these changes

@kaovilai kaovilai kaovilai approved these changes

@eemcmullan eemcmullan Awaiting requested review from eemcmullan

Assignees

@kaovilai kaovilai

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. jira/valid-reference Indicates that this PR references a valid Jira ticket of any type. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@mpryc @openshift-ci-robot @kaovilai @openshift-cherrypick-robot @weshayutin