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

OCM-4442: Moved kmsARN regexp validator to common lib #334

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

den-rgb
Copy link

@den-rgb den-rgb commented Oct 5, 2023

@openshift-ci
Copy link

openshift-ci bot commented Oct 5, 2023

Hi @den-rgb. Thanks for your PR.

I'm waiting for a terraform-redhat member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

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.

@sagidayan sagidayan changed the title OCM-3155 | feat: Moved kmsARN regexp validator to common lib OCM-3155: Moved kmsARN regexp validator to common lib Oct 5, 2023
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 5, 2023

@den-rgb: This pull request references OCM-3155 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 story to target the "4.15.0" version, but no target version was set.

In response to this:

moved kms Arn regexp validator to common lib
Related to : openshift-online/ocm-common#13
JIRA: https://issues.redhat.com/browse/OCM-3155

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.

@sagidayan
Copy link
Contributor

/ok-to-test

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 6, 2023

@den-rgb: This pull request references OCM-3155 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 story to target the "4.15.0" version, but no target version was set.

In response to this:

moved kms Arn regexp validator to common lib
Related to : openshift-online/ocm-common#13
JIRA: https://issues.redhat.com/browse/OCM-3155

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.

@sagidayan
Copy link
Contributor

@den-rgb please rebase your branch to include latest CI changes due to tf1.6.0 breaking changes.

Also before pushing, run make tests and ensure they all pass

@den-rgb
Copy link
Author

den-rgb commented Oct 6, 2023

make tests

I need to wait for ocm-common changes to be merged first before I can continue with this

@nirarg
Copy link
Member

nirarg commented Oct 9, 2023

@den-rgb thank you for this contribution
I'm putting this PR on hold because next version v1.4.0 is in code freeze
I will release the hold once the main branch will open to v1.5.0
/hold

@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 9, 2023

@den-rgb: This pull request references OCM-3155 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 story to target the "4.15.0" version, but no target version was set.

In response to this:

moved kms Arn regexp validator to common lib
Related to : openshift-online/ocm-common#13
JIRA: https://issues.redhat.com/browse/OCM-3155

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.

@den-rgb den-rgb force-pushed the OCM-3155 branch 5 times, most recently from a38d5e1 to af0c2e4 Compare October 23, 2023 10:24
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 23, 2023

@den-rgb: This pull request references OCM-3155 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 story to target the "4.15.0" version, but no target version was set.

In response to this:

moved kms Arn regexp validator to common lib
Related to : openshift-online/ocm-common#13
JIRA: https://issues.redhat.com/browse/OCM-3155, https://issues.redhat.com/browse/OCM-4442

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.

@den-rgb den-rgb changed the title OCM-3155: Moved kmsARN regexp validator to common lib OCM-4442: Moved kmsARN regexp validator to common lib Oct 23, 2023
@openshift-ci-robot
Copy link

openshift-ci-robot commented Oct 23, 2023

@den-rgb: This pull request references OCM-4442 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 task to target the "4.15.0" version, but no target version was set.

In response to this:

moved kms Arn regexp validator to common lib
Related to : openshift-online/ocm-common#13
JIRA: https://issues.redhat.com/browse/OCM-3155, https://issues.redhat.com/browse/OCM-4442

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.

@den-rgb den-rgb force-pushed the OCM-3155 branch 2 times, most recently from 3656846 to d471620 Compare October 27, 2023 11:00
@den-rgb den-rgb requested a review from bardielle October 27, 2023 11:01
@nirarg
Copy link
Member

nirarg commented Oct 29, 2023

/hold cancel
The main branch is alredy opened for v1.5.0
@den-rgb Do you think it ready for merge?

@den-rgb
Copy link
Author

den-rgb commented Oct 31, 2023

/hold cancel The main branch is alredy opened for v1.5.0 @den-rgb Do you think it ready for merge?

@nirarg Yes It should be ready to merge, I'll just rebase it just in case and if everything is alright then it should be good to go

@sagidayan
Copy link
Contributor

/approve

@bardielle
Copy link
Member

/lgtm

@den-rgb
Copy link
Author

den-rgb commented Oct 31, 2023

Could someone point me to how to resolve these errors as they don't seem to be related to my changes, Thanks

@bardielle
Copy link
Member

can you rebace on the main branch

@sagidayan
Copy link
Contributor

/approve

Thanks @den-rgb 👏🏽

Copy link

openshift-ci bot commented Nov 2, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sagidayan

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

@bardielle
Copy link
Member

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Nov 2, 2023
@openshift-ci openshift-ci bot merged commit da8a10a into terraform-redhat:main Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants