Skip to content
This repository was archived by the owner on Jun 26, 2024. It is now read-only.

Expose binding secret without requiring a workload#1206

Merged
openshift-merge-robot merged 1 commit intoredhat-developer:masterfrom
sadlerap:early-binding-name
Aug 25, 2022
Merged

Expose binding secret without requiring a workload#1206
openshift-merge-robot merged 1 commit intoredhat-developer:masterfrom
sadlerap:early-binding-name

Conversation

@sadlerap
Copy link
Contributor

Signed-off-by: Andy Sadler [email protected]

Changes

/kind enhancement

This exposes the relevant binding secret without requiring a workload to be present.

Submitter Checklist

As the author of this PR, please check off the items in this checklist:

  • Docs
    included if any changes are user facing
  • Tests
    included if any functionality added or changed. For bugfixes please include tests that can catch regressions
  • All acceptance test scenarios included in the PR which verifies a bugfix or a requested feature reported by a non-member are tagged with @external-feedback tag.
  • Follows the commit message standard

@openshift-ci openshift-ci bot added the kind/enhancement New feature or request label Aug 22, 2022
@openshift-ci openshift-ci bot requested review from baijum and shruthihub August 22, 2022 22:00
@codecov
Copy link

codecov bot commented Aug 23, 2022

Codecov Report

Merging #1206 (0e9c51e) into master (8622634) will increase coverage by 0.08%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1206      +/-   ##
==========================================
+ Coverage   57.96%   58.05%   +0.08%     
==========================================
  Files          33       33              
  Lines        2838     2844       +6     
==========================================
+ Hits         1645     1651       +6     
  Misses       1032     1032              
  Partials      161      161              
Impacted Files Coverage Δ
pkg/reconcile/pipeline/api.go 77.77% <ø> (ø)
pkg/reconcile/pipeline/context/impl.go 66.24% <100.00%> (+0.17%) ⬆️
pkg/reconcile/pipeline/handler/project/impl.go 54.21% <100.00%> (+0.98%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8622634...0e9c51e. Read the comment docs.

@sadlerap sadlerap force-pushed the early-binding-name branch 2 times, most recently from c194867 to 33aa224 Compare August 23, 2022 20:49
@sadlerap sadlerap force-pushed the early-binding-name branch from 33aa224 to 0e9c51e Compare August 23, 2022 20:57
Copy link
Contributor

@baijum baijum left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Aug 24, 2022
@baijum baijum requested a review from pmacik August 24, 2022 14:51
@sadlerap
Copy link
Contributor Author

/retest

@pmacik
Copy link
Contributor

pmacik commented Aug 25, 2022

/lgtm
/approve

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Aug 25, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pmacik

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

The pull request process is described here

Details 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

@pmacik pmacik added this to the 1.2.0 milestone Aug 25, 2022
@sadlerap
Copy link
Contributor Author

/retest

@openshift-merge-robot openshift-merge-robot merged commit b8086ca into redhat-developer:master Aug 25, 2022
@sadlerap sadlerap deleted the early-binding-name branch August 26, 2022 13:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants