ACM-27053 Remove validation for generators in the ApplicationSet YAML editor#5471
Conversation
… editor Signed-off-by: John Swanke <jswanke@redhat.com>
|
/hold |
|
@jeswanke Thanks for validating all the generators! The code looks good. One issue is that we still get a validation error about the missing "clusterDecisionResource" generator when using matrix generators. Can you remove this validation error as well? Thanks! |
Signed-off-by: John Swanke <jswanke@redhat.com>
| import { CreateArgoResources } from './CreateArgoResources' | ||
| import { MultipleSourcesSelector } from './MultipleSourcesSelector' | ||
| import { SourceSelector } from './SourceSelector' | ||
| import { useYamlResources } from '../../routes/Applications/Applications' |
There was a problem hiding this comment.
Looks like this function doesn't exist:
…-27053-Remove-validation-for-generators-in-the-ApplicationSet-YAML-editor Signed-off-by: John Swanke <jswanke@redhat.com>
Signed-off-by: John Swanke <jswanke@redhat.com>
Signed-off-by: John Swanke <jswanke@redhat.com>
|
@jeswanke I think overall the code looks great! The only minor nit I have is to make the generator inputs show the placeholder text instead of a sample input value. You can even handle this in a bug as well. Thanks! |
|
/lgtm |
|
/retest |
…-27053-Remove-validation-for-generators-in-the-ApplicationSet-YAML-editor Signed-off-by: John Swanke <jswanke@redhat.com>
Signed-off-by: John Swanke <jswanke@redhat.com>
Signed-off-by: fxiang1 <fxiang@redhat.com>
Add back changes from PR 5508
…cationSet-YAML-editor' of https://github.com/jeswanke/console into ACM-27053-Remove-validation-for-generators-in-the-ApplicationSet-YAML-editor Signed-off-by: John Swanke <jswanke@redhat.com>
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fxiang1, jeswanke The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
|
@jeswanke: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions 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-sigs/prow repository. I understand the commands that are listed here. |
… editor (stolostron#5471) * ACM-27053 Remove validation for generators in the ApplicationSet YAML editor Signed-off-by: John Swanke <jswanke@redhat.com> * ACM-27053 Support all Argo CD generators in the ApplicationSet Wizard Signed-off-by: John Swanke <jswanke@redhat.com> * backup Signed-off-by: John Swanke <jswanke@redhat.com> * backup Signed-off-by: John Swanke <jswanke@redhat.com> * backup Signed-off-by: John Swanke <jswanke@redhat.com> * backup Signed-off-by: John Swanke <jswanke@redhat.com> * fix tests Signed-off-by: John Swanke <jswanke@redhat.com> * fix sync bug Signed-off-by: John Swanke <jswanke@redhat.com> * fix checks Signed-off-by: John Swanke <jswanke@redhat.com> * prettier Signed-off-by: John Swanke <jswanke@redhat.com> * oops Signed-off-by: John Swanke <jswanke@redhat.com> * oop[s Signed-off-by: John Swanke <jswanke@redhat.com> * oops Signed-off-by: John Swanke <jswanke@redhat.com> * oops Signed-off-by: John Swanke <jswanke@redhat.com> * fix lint Signed-off-by: John Swanke <jswanke@redhat.com> * fix coverage Signed-off-by: John Swanke <jswanke@redhat.com> * Put back changes from PR 5508 Signed-off-by: fxiang1 <fxiang@redhat.com> * linjt Signed-off-by: John Swanke <jswanke@redhat.com> * fix test Signed-off-by: John Swanke <jswanke@redhat.com> --------- Signed-off-by: John Swanke <jswanke@redhat.com> Signed-off-by: fxiang1 <fxiang@redhat.com> Co-authored-by: fxiang1 <fxiang@redhat.com>
3 participants
Signed-off-by: John Swanke jswanke@redhat.com
📝 Summary
Ticket Summary (Title):
Ticket Link:
Type of Change:
✅ Checklist
General
ACM-12340 Fix bug with...)If Feature
If Bugfix
🗒️ Notes for Reviewers