Skip to content

Conversation

juanvallejo
Copy link
Contributor

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1544183

  • Implements the ExistenceChecker interface for Deployments and ReplicaSets
    which allows the HPA error marker to be set if it is trying to scale a deleted Deployment (or rs).

  • Skips standalone resources that are not found

cc @soltysh @mfojtik @deads2k

@openshift-ci-robot openshift-ci-robot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 12, 2018
@juanvallejo juanvallejo force-pushed the jvallejo/fix-standalone-deployments branch from 1fb4271 to 6e1a34e Compare February 12, 2018 19:20
@juanvallejo juanvallejo changed the title Updates to Deployments in status graph Deployment updates in status graph Feb 12, 2018
@juanvallejo
Copy link
Contributor Author

/test extended_conformance_install

1 similar comment
@juanvallejo
Copy link
Contributor Author

/test extended_conformance_install

@mfojtik
Copy link
Contributor

mfojtik commented Feb 13, 2018

/lgtm

thanks!

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 13, 2018
@openshift-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: juanvallejo, mfojtik

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these OWNERS Files:

You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancel in a comment

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 13, 2018
@openshift-merge-robot
Copy link
Contributor

Automatic merge from submit-queue (batch tested with PRs 18437, 18546, 18550, 18579).

@openshift-merge-robot openshift-merge-robot merged commit 689dc7d into openshift:master Feb 13, 2018
@juanvallejo juanvallejo deleted the jvallejo/fix-standalone-deployments branch February 13, 2018 15:11
openshift-merge-robot added a commit that referenced this pull request Feb 26, 2018
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. lgtm Indicates that a PR is ready to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants