Skip to content

Conversation

@Daniel-Fan
Copy link
Contributor

No description provided.

Daniel-Fan and others added 20 commits July 19, 2021 23:10
Bumps golang from 1.15.7 to 1.16.6.

---
updated-dependencies:
- dependency-name: golang
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* refactor ODLM cache to use one pattern of design

* update another cache design for e2e test
Bumps golang from 1.16.6 to 1.16.7.

---
updated-dependencies:
- dependency-name: golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* opt-in of ubi base image automated builds

* configure Docker Credential
* add relatedImages to operator CSV

* follow CICD name convention
* make operatorconfig optional

* update the err msg & err check
Bumps golang from 1.16.7 to 1.17.0.

---
updated-dependencies:
- dependency-name: golang
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* build: bump k8s dependencies to 1.21

* Download tools locally

* disable package verification

* unset GOSUMDB
…#737)

Current logic makes the operator uninstall by console/cli reinstall immediately, which is not expected by all the users
* fix: Operator only reconciles when watched subscriptions are updated.

Current logic makes the operator uninstall by console/cli reinstall immediately, which is not expected by all the users

* feat: Enable subscription config option in the operandregistry
@codecov
Copy link

codecov bot commented Sep 1, 2021

Codecov Report

Merging #741 (20dadf4) into release-future (bce667c) will increase coverage by 0.55%.
The diff coverage is 52.77%.

Impacted file tree graph

@@                Coverage Diff                 @@
##           release-future     #741      +/-   ##
==================================================
+ Coverage           65.85%   66.40%   +0.55%     
==================================================
  Files                  10       10              
  Lines                1678     1685       +7     
==================================================
+ Hits                 1105     1119      +14     
+ Misses                388      385       -3     
+ Partials              185      181       -4     
Impacted Files Coverage Δ
controllers/operandrequest/reconcile_operand.go 59.45% <25.00%> (-1.09%) ⬇️
controllers/operandrequest/reconcile_operator.go 63.90% <61.90%> (+0.40%) ⬆️
...ollers/operandrequest/operandrequest_controller.go 74.05% <100.00%> (+4.37%) ⬆️
...lers/operandbindinfo/operandbindinfo_controller.go 76.96% <0.00%> (-2.92%) ⬇️
...trollers/operandconfig/operandconfig_controller.go 73.28% <0.00%> (+8.39%) ⬆️
...lers/operandregistry/operandregistry_controller.go 84.00% <0.00%> (+10.00%) ⬆️

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 bce667c...20dadf4. Read the comment docs.

Copy link
Contributor

@horis233 horis233 left a comment

Choose a reason for hiding this comment

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

/lgtm

@ibm-ci-bot ibm-ci-bot added the lgtm label Sep 1, 2021
@ibm-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Daniel-Fan, horis233

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:
  • OWNERS [Daniel-Fan,horis233]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Daniel-Fan
Copy link
Contributor Author

/retest

@ibm-ci-bot ibm-ci-bot merged commit 39949cf into release-future Sep 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants