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

feat: added env variable to skip gitops validation on create/update #3956

Merged

Ash-exp merged 5 commits into main from feat-skip-gitops-validation

Sep 25, 2023

Contributor

Ash-exp commented Sep 22, 2023 •

edited

Description

Added an env variable to skip GitOps validation in Global configuration.

Fixes #3961

How Has This Been Tested?

Checklist:

The title of the PR states what changed and the related issues number (used for the release note).
Does this PR requires documentation updates?
I've updated documentation as required by this PR.
I have performed a self-review of my own code.
I have commented my code, particularly in hard-to-understand areas.
I have tested it for all user roles.
I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?


          feat: added env variable to skip gitops validation on create/update

26081d8

Ash-exp added the enhancement label

Ash-exp requested review from iamayushm and vikramdevtron

September 22, 2023 12:15

Ash-exp self-assigned this

prakarsh-dt added the PR:Issue-verification-failed label

Ash-exp added 2 commits

September 22, 2023 18:35


          feat: updated API response

00fb828


          feat: reverted API response changes

6e1463b

vikramdevtron previously approved these changes

View reviewed changes


          handled panic on CD pipeline save

457242d

Ash-exp dismissed vikramdevtron’s stale review via

457242d

September 25, 2023 06:57

vikramdevtron approved these changes

View reviewed changes


          Merge branch 'main' into feat-skip-gitops-validation

799a6aa

sonarcloud bot commented Sep 25, 2023

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

prakarsh-dt added PR:Ready-to-Review and removed PR:Issue-verification-failed labels

Ash-exp merged commit cd1c4d3 into main

5 of 6 checks passed

Ash-exp deleted the feat-skip-gitops-validation branch

September 25, 2023 07:38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment