odo icon indicating copy to clipboard operation
odo copied to clipboard

Update odo to use go 1.18

Open dharmit opened this issue 3 years ago • 16 comments

Signed-off-by: Dharmit Shah [email protected]

What type of PR is this: /kind feature

What does this PR do / why we need it: $subject

Which issue(s) this PR fixes:

Fixes #6090

PR acceptance criteria:

  • [ ] Unit test

  • [ ] Integration test

  • [ ] Documentation

How to test changes / Special notes to the reviewer:

  • I tested make validate and make test locally before opening the PR. Not sure if there was any other Makefile target that I should have tested locally.
  • For a list of steps I followed, PTAL https://github.com/redhat-developer/odo/issues/6090#issuecomment-1255199161.

dharmit avatar Sep 23 '22 13:09 dharmit

Deploy Preview for odo-docusaurus-preview ready!

Name Link
Latest commit 50bdddc9e58807ce72c6a6d953f505e521369299
Latest deploy log https://app.netlify.com/sites/odo-docusaurus-preview/deploys/636b5d0f27fd700008a2f25c
Deploy Preview https://deploy-preview-6166--odo-docusaurus-preview.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Sep 23 '22 13:09 netlify[bot]

Unit Tests on commit d3d598675d8e085d4f7c9a4dc392af76b31fca16 finished successfully. View logs: TXT HTML

odo-robot[bot] avatar Sep 23 '22 13:09 odo-robot[bot]

Validate Tests on commit d3d598675d8e085d4f7c9a4dc392af76b31fca16 finished successfully. View logs: TXT HTML

odo-robot[bot] avatar Sep 23 '22 13:09 odo-robot[bot]

Kubernetes Tests on commit d3d598675d8e085d4f7c9a4dc392af76b31fca16 finished successfully. View logs: TXT HTML

odo-robot[bot] avatar Sep 23 '22 13:09 odo-robot[bot]

OpenShift Tests on commit d3d598675d8e085d4f7c9a4dc392af76b31fca16 finished successfully. View logs: TXT HTML

odo-robot[bot] avatar Sep 23 '22 13:09 odo-robot[bot]

Windows Tests (OCP) on commit d3d598675d8e085d4f7c9a4dc392af76b31fca16 finished successfully. View logs: TXT HTML

odo-robot[bot] avatar Sep 23 '22 13:09 odo-robot[bot]

@dharmit, just wanted to know if we have checked with EXD team if they have go 1.18 on rhel8, I remember we have go1.18 available for rhel7.

anandrkskd avatar Sep 23 '22 13:09 anandrkskd

@dharmit, just wanted to know if we have checked with EXD team if they have go 1.18 on rhel8, I remember we have go1.18 available for rhel7.

I have pinged Matej for this. I checked using the brew cli, but couldn't get the answer myself.

dharmit avatar Oct 03 '22 04:10 dharmit

I have pinged Matej for this. I checked using the brew cli, but couldn't get the answer myself.

I checked for 1.18 on rhel8 but was not able to find it. for rhel 7 its available AFAIR

anandrkskd avatar Oct 03 '22 05:10 anandrkskd

I have pinged Matej for this. I checked using the brew cli, but couldn't get the answer myself.

I checked for 1.18 on rhel8 but was not able to find it. for rhel 7 its available AFAIR

Indeed. That's exactly what I found. But I saw the list had only up to go 1.10, which seems weird to me because it's too old. That's why I have pinged Matej about it since he would know the best. :)

dharmit avatar Oct 03 '22 06:10 dharmit

@dharmit All tests fail, did you rebase after this PR? https://github.com/redhat-developer/odo/pull/6164

feloy avatar Oct 03 '22 07:10 feloy

@dharmit All tests fail, did you rebase after this PR? #6164

I don't remember if I did or didn't. Anyway, rebased now.

dharmit avatar Oct 03 '22 11:10 dharmit

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Oct 03 '22 11:10 sonarqubecloud[bot]

/hold

Putting this on hold to prevent accidental merge till Red Hat Release Engg team says OK to update to 1.18. In the meantime, I'll try to address CI issues and the folks can review/approve it so that it can be merged once we have a go ahead.

dharmit avatar Oct 03 '22 12:10 dharmit

NoCluster Tests on commit d3d598675d8e085d4f7c9a4dc392af76b31fca16 finished successfully. View logs: TXT HTML

odo-robot[bot] avatar Nov 04 '22 10:11 odo-robot[bot]

/hold

Putting this on hold to prevent accidental merge till Red Hat Release Engg team says OK to update to 1.18. In the meantime, I'll try to address CI issues and the folks can review/approve it so that it can be merged once we have a go ahead.

/hold cancel

Internal systems at RH have been updated to use go 1.18.

dharmit avatar Nov 04 '22 10:11 dharmit

/approve

feloy avatar Nov 04 '22 16:11 feloy

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: feloy

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[bot] avatar Nov 04 '22 16:11 openshift-ci[bot]

Both failures, NoCluster & OCP prow tests, are attributed to the problem mentioned in #6291 and fixed by #6292.

dharmit avatar Nov 08 '22 07:11 dharmit

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar Nov 09 '22 07:11 sonarqubecloud[bot]

<< End Captured GinkgoWriter Output
  Unexpected error:
      <*url.Error | 0xc000170120>: {
          Op: "Get",
          URL: "http://127.0.0.1:45923/",
          Err: <*errors.errorString | 0xc0000b0100>{s: "EOF"},
      }
      Get "http://127.0.0.1:45923/": EOF
  occurred
  In [It] at: /go/src/github.com/redhat-developer/odo/tests/e2escenarios/e2e_test.go:32
------------------------------

overriding since tests are passing on IBM Cloud

/override ci/prow/v4.11-integration-e2e

dharmit avatar Nov 09 '22 10:11 dharmit

@dharmit: Overrode contexts on behalf of dharmit: ci/prow/v4.11-integration-e2e

In response to this:

<< End Captured GinkgoWriter Output
 Unexpected error:
     <*url.Error | 0xc000170120>: {
         Op: "Get",
         URL: "http://127.0.0.1:45923/",
         Err: <*errors.errorString | 0xc0000b0100>{s: "EOF"},
     }
     Get "http://127.0.0.1:45923/": EOF
 occurred
 In [It] at: /go/src/github.com/redhat-developer/odo/tests/e2escenarios/e2e_test.go:32
------------------------------

overriding since tests are passing on IBM Cloud

/override ci/prow/v4.11-integration-e2e

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

openshift-ci[bot] avatar Nov 09 '22 10:11 openshift-ci[bot]