Skip to content

Rollback go and controller runtime versions#16

Merged
thepetk merged 4 commits intodevfile:bck/1.1.0from
thepetk:backported_1.1.0
Aug 4, 2023
Merged

Rollback go and controller runtime versions#16
thepetk merged 4 commits intodevfile:bck/1.1.0from
thepetk:backported_1.1.0

Conversation

@thepetk
Copy link
Contributor

@thepetk thepetk commented Aug 4, 2023

What does this PR do?

This PR rolls back go to version 1.18 and controller-runtime to v0.14.4.

We will not be using the main branch for this version, instead we will be using the bck/1.1.0.

Once the application-api updates its version of go to 1.19 we will deprecate the backported tag.

Which issue(s) does this PR fix

fixes devfile/api#1210

PR acceptance criteria

Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened.

  • Unit/Functional tests

  • Documentation

How to test changes / Special notes to the reviewer

@thepetk thepetk requested review from mike-hoang and yangcao77 August 4, 2023 11:15
@thepetk thepetk self-assigned this Aug 4, 2023
@thepetk thepetk changed the base branch from bck/1.1.0 to main August 4, 2023 11:16
@thepetk thepetk changed the title Rollback go and controller runtime versions [DO NOT MERGE] Rollback go and controller runtime versions Aug 4, 2023
@thepetk
Copy link
Contributor Author

thepetk commented Aug 4, 2023

I've created the PR temporarily to point to main in order to check if all actions are passing. After everything all tests are passing I'll point back to bck/1.1.0

@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (aa486b0) 66.56% compared to head (00b28d4) 66.56%.

Additional details and impacted files
@@            Coverage Diff             @@
##           bck/1.1.0      #16   +/-   ##
==========================================
  Coverage      66.56%   66.56%           
==========================================
  Files             10       10           
  Lines           1358     1358           
==========================================
  Hits             904      904           
  Misses           394      394           
  Partials          60       60           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@thepetk thepetk changed the base branch from main to bck/1.1.0 August 4, 2023 11:28
@thepetk thepetk changed the title [DO NOT MERGE] Rollback go and controller runtime versions Rollback go and controller runtime versions Aug 4, 2023
@thepetk
Copy link
Contributor Author

thepetk commented Aug 4, 2023

I've created the PR temporarily to point to main in order to check if all actions are passing. After everything all tests are passing I'll point back to bck/1.1.0

All tests are passing - I've pointed back to bck/1.1.0 and the PR is ready for review

@openshift-ci
Copy link

openshift-ci bot commented Aug 4, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: mike-hoang, thepetk

The full list of commands accepted by this bot can be found 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

@thepetk thepetk merged commit 14d5e46 into devfile:bck/1.1.0 Aug 4, 2023
thepetk referenced this pull request in thepetk/devfile-alizer Sep 20, 2023
* Rollback go and controller runtime versions

Signed-off-by: thepetk <[email protected]>

* Update workflows versions

Signed-off-by: thepetk <[email protected]>

* Add warning for backported version

Signed-off-by: thepetk <[email protected]>

* Rollback version of multierr

Signed-off-by: thepetk <[email protected]>

---------

Signed-off-by: thepetk <[email protected]>
thepetk added a commit that referenced this pull request Sep 20, 2023
* Rollback go and controller runtime versions (#16)

* Rollback go and controller runtime versions

Signed-off-by: thepetk <[email protected]>

* Update workflows versions

Signed-off-by: thepetk <[email protected]>

* Add warning for backported version

Signed-off-by: thepetk <[email protected]>

* Rollback version of multierr

Signed-off-by: thepetk <[email protected]>

---------

Signed-off-by: thepetk <[email protected]>

* Update go version to 1.19

Signed-off-by: thepetk <[email protected]>

---------

Signed-off-by: thepetk <[email protected]>
thepetk added a commit that referenced this pull request Oct 4, 2023
* Rollback go and controller runtime versions (#16)

* Rollback go and controller runtime versions

Signed-off-by: thepetk <[email protected]>

* Update workflows versions

Signed-off-by: thepetk <[email protected]>

* Add warning for backported version

Signed-off-by: thepetk <[email protected]>

* Rollback version of multierr

Signed-off-by: thepetk <[email protected]>

---------

Signed-off-by: thepetk <[email protected]>

* Update go version to 1.19

Signed-off-by: thepetk <[email protected]>

---------

Signed-off-by: thepetk <[email protected]>
thepetk added a commit that referenced this pull request Oct 4, 2023
* Rollback go and controller runtime versions (#16)

* Rollback go and controller runtime versions

Signed-off-by: thepetk <[email protected]>

* Update workflows versions

Signed-off-by: thepetk <[email protected]>

* Add warning for backported version

Signed-off-by: thepetk <[email protected]>

* Rollback version of multierr

Signed-off-by: thepetk <[email protected]>

---------

Signed-off-by: thepetk <[email protected]>

* Update go version to 1.19

Signed-off-by: thepetk <[email protected]>

---------

Signed-off-by: thepetk <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Create backported version of Alizer 1.1.1 to use fixed dependencies and go versions

2 participants