Skip to content

chore(orc8r): Upgrade go to 1.20.1#15134

Merged
voisey merged 5 commits intomasterfrom
go_upgrade
Mar 16, 2023
Merged

chore(orc8r): Upgrade go to 1.20.1#15134
voisey merged 5 commits intomasterfrom
go_upgrade

Conversation

@MoritzThomasHuebner
Copy link
Copy Markdown
Contributor

@MoritzThomasHuebner MoritzThomasHuebner commented Mar 7, 2023

@pull-request-size pull-request-size bot added the size/M Denotes a PR that changes 30-99 lines. label Mar 7, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 7, 2023

Thanks for opening a PR! 💯

A couple initial guidelines

Howto

  • Reviews. The "Reviewers" listed for this PR are the Magma maintainers who will shepherd it.
  • Checks. All required CI checks must pass before merge.
  • Merge. Once approved and passing CI checks, use the ready2merge label to indicate the maintainers can merge your PR.

More info

Please take a moment to read through the Magma project's

If this is your first Magma PR, also consider reading

@github-actions github-actions bot added component: agw Access gateway-related issue component: ci All updates on CI (Jenkins/CircleCi/Github Action) component: cwf component: docs Documentation-related issue component: feg FEG-gateway related issues component: orc8r Orchestrator-related issue labels Mar 7, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 7, 2023

✔️ The Semantic PR check ended with status success. See instructions on formatting your commit and pull request titles.

@MoritzThomasHuebner MoritzThomasHuebner changed the title Go upgrade chore(orc8r): Upgrade go to 1.20.1 Mar 7, 2023
@MoritzThomasHuebner MoritzThomasHuebner linked an issue Mar 7, 2023 that may be closed by this pull request
@pull-request-size pull-request-size bot added size/L Denotes a Pull Request that changes 100-499 lines. and removed size/M Denotes a PR that changes 30-99 lines. labels Mar 7, 2023
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 7, 2023

CWF integration test results

  1 files  ±0    1 suites  ±0   16m 42s ⏱️ + 5m 50s
40 tests ±0  22 ✔️  - 6  11 💤 ±0  7 +6 
48 runs  +7  29 ✔️ ±0  11 💤 ±0  8 +7 

For more details on these failures, see this check.

Results for commit 67a6fac. ± Comparison against base commit 6368195.

♻️ This comment has been updated with latest results.

@github-actions github-actions bot removed the component: agw Access gateway-related issue label Mar 7, 2023
@MoritzThomasHuebner MoritzThomasHuebner marked this pull request as ready for review March 7, 2023 09:18
@MoritzThomasHuebner MoritzThomasHuebner requested review from a team as code owners March 7, 2023 09:18
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 8, 2023

FEG integration test results

32 tests  ±0   32 ✔️ ±0   22m 55s ⏱️ + 2m 50s
32 suites ±0     0 💤 ±0 
32 files   ±0     0 ±0 

Results for commit 6f47b6b. ± Comparison against base commit 503c706.

MoritzThomasHuebner and others added 2 commits March 15, 2023 11:27
Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>
ci, docker, ansible

Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>
MoritzThomasHuebner and others added 3 commits March 15, 2023 11:27
Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>
Needed for go1.20

Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>
Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>
Copy link
Copy Markdown
Member

@maxhbr maxhbr left a comment

Choose a reason for hiding this comment

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

LGTM

@voisey voisey merged commit e52f582 into master Mar 16, 2023
@voisey voisey deleted the go_upgrade branch March 16, 2023 13:43
lucasgonze pushed a commit to lucasgonze/magma that referenced this pull request Feb 29, 2024
* chore(orc8r): Increment go version

Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>

* chore(orc8r): Increment go version in configs

ci, docker, ansible

Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>

* chore(orc8r): Increment go version in go.mod

Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>

* chore(orc8r): Upgrade golangci-lint

Needed for go1.20

Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>

* chore(orc8r): Changed golang versions to 1.20 where missing

Co-authored-by: Sebastian Wolf <[email protected]>
Signed-off-by: Moritz Huebner <[email protected]>

---------

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

Labels

component: ci All updates on CI (Jenkins/CircleCi/Github Action) component: cwf component: docs Documentation-related issue component: feg FEG-gateway related issues component: orc8r Orchestrator-related issue size/L Denotes a Pull Request that changes 100-499 lines.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade Go to version 1.20

3 participants