Skip to content

Conversation

losipiuk
Copy link
Contributor

@losipiuk losipiuk commented Feb 4, 2020

Final version of PR migrating PredicateChecker to scheduler framework.

Review already done in #2709.

List of outstanding items to be addressed as followups: #2797

…er (3fca0a6)

Change-Id: Id9b4508a870fbf89ef78713ea9b89a046bf2a445
It seems it is not needed any more
@k8s-ci-robot k8s-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Feb 4, 2020
@k8s-ci-robot k8s-ci-robot added the cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. label Feb 4, 2020
@losipiuk losipiuk force-pushed the lo/scheduler-framework-poc3 branch from af23ae7 to 4b30a6f Compare February 4, 2020 19:54
This is temporary hack. We should stop using for-go-proj.sh to
verify cluster autoscaler and migrate to directly calling make.

Change-Id: I0c41c918f78cdfa0e5fd0dd60139a54db90540ea
@MaciekPytel MaciekPytel added area/cluster-autoscaler area/core-autoscaler Denotes an issue that is related to the core autoscaler and is not specific to any provider. labels Feb 5, 2020
@MaciekPytel
Copy link
Contributor

MaciekPytel commented Feb 5, 2020

/lgtm
/approve

edit: obviously it will really only 'look good' after follow-ups are addressed, but it's not sustainable to keep working on this in a feature branch.

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 5, 2020
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: MaciekPytel

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 5, 2020
@k8s-ci-robot k8s-ci-robot merged commit 66f6328 into kubernetes:master Feb 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. area/cluster-autoscaler area/core-autoscaler Denotes an issue that is related to the core autoscaler and is not specific to any provider. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants