Skip to content
This repository has been archived by the owner on Nov 11, 2024. It is now read-only.

Don't fail when adding a service to an environment with no pipelines. #107

Merged
merged 1 commit into from
Jan 26, 2021

Conversation

bigkevmcd
Copy link
Contributor

What type of PR is this?

/kind bug

What does this PR do / why we need it:

Don't fail when you provide a service with an image to an environment with no pipelines

Have you updated the necessary documentation?

  • Documentation update is required by this PR.
  • Documentation has been updated.

Which issue(s) this PR fixes:

Fixes #?

#102

How to test changes / Special notes to the reviewer:

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Dec 3, 2020
@openshift-ci-robot
Copy link
Collaborator

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@wtam2018
Copy link
Collaborator

wtam2018 commented Dec 7, 2020

/lgtm
/approve

@openshift-ci-robot openshift-ci-robot added lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Dec 7, 2020
@wtam2018
Copy link
Collaborator

wtam2018 commented Dec 7, 2020

/lgtm cancel
/approve cancel

@openshift-ci-robot openshift-ci-robot removed lgtm Indicates that a PR is ready to be merged. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Dec 7, 2020
@bigkevmcd bigkevmcd force-pushed the handle-no-pipelines branch 2 times, most recently from 570f307 to 3f10b4a Compare January 5, 2021 13:52
@bigkevmcd bigkevmcd force-pushed the handle-no-pipelines branch from 3f10b4a to a13f5e9 Compare January 18, 2021 08:28
@bigkevmcd bigkevmcd marked this pull request as ready for review January 18, 2021 08:32
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jan 18, 2021
@bigkevmcd
Copy link
Contributor Author

@sbose78 Are you ok to go ahead with this?

@wtam2018
Copy link
Collaborator

/lgtm
/approve

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jan 26, 2021
@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: wtam2018

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

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 26, 2021
@openshift-merge-robot openshift-merge-robot merged commit 09c3f0b into master Jan 26, 2021
@bigkevmcd bigkevmcd deleted the handle-no-pipelines branch February 8, 2021 11:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants