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

PLT-1092 #2886

Merged
merged 3 commits into from
Mar 21, 2024
Merged

PLT-1092 #2886

merged 3 commits into from
Mar 21, 2024

Conversation

hr2904
Copy link
Collaborator

@hr2904 hr2904 commented Mar 21, 2024

Change description

Currently when a tag is propagated, it can be propagated to via a couple of paths
Via hierarchy
Via lineage

When it comes to configure & customize these propagation paths, we only support restricting though lineage with restrictPropagationThroughLineage flag. This means propagations via hierarchy are not optional & done forcefully even when not required.

To make it optional, we have introduced another flag which help to control propagations via hierarchy : restrictPropagationThroughHierarchy
Added Argument of restrictThroughHierarchy in all methods where restrictThroughLineage was being passed, and handled the edge case of both flag being true. Added the concept of toExclude for : flag basis getting of labels.
Handled all test cases and made fixes for them.
Added a method to test propagation config before sending it to Background task queue.

Type of change

  • [] New feature (adds functionality)

Related issues

Fix #1

Checklists

Development

  • Lint rules pass locally
  • Application changes have been tested thoroughly
  • Automated tests covering modified code pass

Security

  • Security impact of change has been considered
  • Code follows company security practices and guidelines

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
  • "Ready for review" label attached and reviewers assigned
  • Changes have been reviewed by at least one other contributor
  • Pull request linked to task tracker where applicable

hr2904 added 3 commits March 20, 2024 11:18
…ictThroughLineage was being passed, and handled the edge case of both flag being true. Added the concept of toExclude for flag basis getting of labels.
Added a method to test propagation config before sending it to BG task queue.
@hr2904 hr2904 changed the base branch from master to beta March 21, 2024 10:04
@hr2904 hr2904 merged commit 219a123 into beta Mar 21, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant