Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

[KOGITO-3979] - Switch to current testing suite by OLM #739

Merged
merged 1 commit into from Feb 1, 2021

Conversation

Kaitou786
Copy link
Contributor

@Kaitou786 Kaitou786 commented Jan 27, 2021

See: https://issues.redhat.com/browse/KOGITO-3979

Signed-off-by: Tarun Khandelwal tarkhand@redhat.com

Many thanks for submitting your Pull Request ❤️!

Please make sure your PR meets the following requirements:

  • You have read the contributors' guide
  • Pull Request title is properly formatted: [KOGITO-XYZ] Subject
  • Pull Request contains a link to the JIRA issue
  • Pull Request contains a description of the issue
  • Pull Request does not include fixes for issues other than the main ticket
  • Your feature/bug fix has a unit test that verifies it
  • You've tested the new feature/bug fix in an actual OpenShift cluster
  • You've added a RELEASE_NOTES.md entry regarding this change

@kie-ci-bot kie-ci-bot bot added the WIP 👷‍♂️ Pull Request is on work in progress label Jan 27, 2021
@Kaitou786 Kaitou786 marked this pull request as ready for review January 27, 2021 16:57
@kie-ci-bot kie-ci-bot bot removed the WIP 👷‍♂️ Pull Request is on work in progress label Jan 27, 2021
@kie-ci-bot kie-ci-bot bot added the needs review 🔍 Pull Request that needs reviewers label Jan 27, 2021
@Kaitou786 Kaitou786 added the CI/CD ➿ Pull Request related to Continous Integration/ Deployment label Jan 27, 2021
Copy link
Member

@ricardozanini ricardozanini left a comment

Choose a reason for hiding this comment

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

Awesome man! MANY THANKS for this. Can you please take a look at the comment regarding the update in the mod file?

go.mod Outdated Show resolved Hide resolved
@Kaitou786
Copy link
Contributor Author

Kaitou786 commented Jan 28, 2021

Introduced go.tools.mod =D

@ricardozanini
Copy link
Member

@radtriste, can you take a look? I'd like to merge this before reviewing #740

hack/ci/ensure-image.sh Outdated Show resolved Hide resolved
hack/ci/operator-ensure-manifests.sh Show resolved Hide resolved
hack/ci/operator-ensure-manifests.sh Outdated Show resolved Hide resolved
@Kaitou786 Kaitou786 added ready 🚀 PR is ready to be merged and removed needs review 🔍 Pull Request that needs reviewers labels Jan 29, 2021
@kie-ci-bot kie-ci-bot bot added the needs review 🔍 Pull Request that needs reviewers label Jan 30, 2021
@Kaitou786
Copy link
Contributor Author

Kaitou786 commented Jan 30, 2021

Just squashed the commits :)

@Kaitou786 Kaitou786 removed the needs review 🔍 Pull Request that needs reviewers label Jan 30, 2021
@ricardozanini ricardozanini merged commit c91945f into apache:master Feb 1, 2021
@ricardozanini ricardozanini deleted the KOGITO-3979 branch February 1, 2021 12:22
@ricardozanini
Copy link
Member

@Kaitou786 if you have time, can you please also do the same for the Nexus Operator? Right now, we are struggling with their CI. This might fix the next release there.

@Kaitou786
Copy link
Contributor Author

Yeah I agree, I was waiting for the PR to be merged before doing this

@Kaitou786
Copy link
Contributor Author

I had a talk with OLM guys and since many other are using GH actions to run local OLM tests, so they are planning to create a GH action for it, that'll be really easy for us to use =D

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI/CD ➿ Pull Request related to Continous Integration/ Deployment ready 🚀 PR is ready to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants