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

[QG Part 1] RANCHER-1002 Integrate Cypress Tests Flow with Jira Ticketing and Slack Notifications in folioCiQualityGates Jenkins Pipeline #434

Open
wants to merge 1 commit into
base: RANCHER-1054
Choose a base branch
from

Conversation

OHaimanov
Copy link
Collaborator

Copy link
Contributor

@eldiiar-duishenaliev eldiiar-duishenaliev left a comment

Choose a reason for hiding this comment

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

Hello Sasha,
In overall looks outstanding!
But, some changes could potentially break existing workflows.
As an example another PR created and pending review for a 100% will be conflicting.
Not a big deal to correct kubectl part on it, but please let's avoid existing libs modification, instead, we could implement versioning, something like:
checkNsExistence
checkNsExistencev2 and so on, what do you think?
Thanks again, quite huge changes and approach switch, which is AWSome :)

@OHaimanov OHaimanov force-pushed the RANCHER-1002 branch 5 times, most recently from 8921478 to afecc40 Compare February 26, 2024 15:42
@OHaimanov OHaimanov changed the base branch from RANCHER-741-Jenkins-Enhancements to RANCHER-1054 February 26, 2024 15:44
@OHaimanov OHaimanov changed the title RANCHER-1002 Integrate Cypress Tests Flow with Jira Ticketing and Slack Notifications in folioCiQualityGates Jenkins Pipeline [QG Part 1] RANCHER-1002 Integrate Cypress Tests Flow with Jira Ticketing and Slack Notifications in folioCiQualityGates Jenkins Pipeline Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants