-
Notifications
You must be signed in to change notification settings - Fork 829
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
sync status of pvc #2070
sync status of pvc #2070
Conversation
/assign @XiShanYongYe-Chang |
https://github.com/chaunceyjiang/karmada/actions/runs/2573428844 我感觉 karmada仓库 的github action 是不是有问题呀,好几次我的仓库的e2e是可以过的,一旦到karmada仓库下就不行了 |
Should be fixed with #2072 |
dbbe3f4
to
d709ac5
Compare
ok |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @chaunceyjiang, I just ask a question, I have no actual application scenario, so I am not sure which fields of status need to be collected. Do you think other fields(such as capacity
, conditions
, etc.) need to be considered?
In my scenario, I only care about |
cd3b7c0
to
b9cd77f
Compare
/assign @Garrybest |
3a9d358
to
27c8a09
Compare
/cc @Garrybest |
Thanks! |
pkg/resourceinterpreter/defaultinterpreter/aggregatestatus_test.go
Outdated
Show resolved
Hide resolved
pkg/resourceinterpreter/defaultinterpreter/aggregatestatus_test.go
Outdated
Show resolved
Hide resolved
27c8a09
to
ffaa05a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to update the documents, isn't it? @XiShanYongYe-Chang |
Yes, it is. Hi @chaunceyjiang, can you help update the document? |
ffaa05a
to
b368c29
Compare
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your quick response.
/lgtm
Hi @chaunceyjiang, have you rebase the master branch? |
What I mean is #2095. |
Signed-off-by: chaunceyjiang <chaunceyjiang@gmail.com>
b368c29
to
6ffe6f0
Compare
Thanks for your mind, it does need to be upgraded. |
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
/approve
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: RainbowMango The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: chaunceyjiang chaunceyjiang@gmail.com
What type of PR is this?
/kind feature
What this PR does / why we need it:
PVC used frequently. I think that build-in interpreter aggregatestatus should contain PVC
Which issue(s) this PR fixes:
Fixes #2065
Special notes for your reviewer:
Does this PR introduce a user-facing change?: