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

Generate resourcebinding from clusterpropagationpolicy based on the "namespace" exist of the coming resource #315

Conversation

vincent-pli
Copy link
Contributor

What type of PR is this?
/kind feature

What this PR does / why we need it:
Since ClusterPropagationPolicy is allowed to point to a namespace scoped resource, so in the scenario a ResourceBinding rather than ClusterResourceBinding should be created.

Which issue(s) this PR fixes:
Fixes #300

Special notes for your reviewer:

Does this PR introduce a user-facing change?:
NONE


@vincent-pli vincent-pli force-pushed the generate-resourcebinding-from-clusterpropagationpolicy branch from f13e98d to 6c93ea3 Compare May 11, 2021 06:32
@RainbowMango RainbowMango self-assigned this May 11, 2021
@RainbowMango
Copy link
Member

Thanks!!! I'll review it later, maybe tomorrow.

Copy link
Member

@RainbowMango RainbowMango left a comment

Choose a reason for hiding this comment

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

Just some nits, otherwise LGTM.

There are some duplicate codes in ApplyClusterPolicy method, but it's no big deal.

pkg/util/detector/detector.go Outdated Show resolved Hide resolved
pkg/util/detector/detector.go Outdated Show resolved Hide resolved
…namespace" exist of the comming object

Signed-off-by: pengli <justdoit.pli@gmail.com>
@vincent-pli vincent-pli force-pushed the generate-resourcebinding-from-clusterpropagationpolicy branch from 6c93ea3 to 9dfa74d Compare May 13, 2021 00:31
@RainbowMango
Copy link
Member

Thanks and congrats! @vincent-pli

@RainbowMango RainbowMango merged commit e0f725c into karmada-io:master May 13, 2021
XiShanYongYe-Chang pushed a commit to XiShanYongYe-Chang/karmada that referenced this pull request Aug 28, 2021
…namespace" exist of the comming object (karmada-io#315)

Signed-off-by: pengli <justdoit.pli@gmail.com>
jwcesign pushed a commit to jwcesign/karmada that referenced this pull request Jun 8, 2023
add CronJob to build-in AggregateStatus Interpreter
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.

[HelpWanted] ClusterResourceBinding should only reference cluster-scoped resources
2 participants