Skip to content

Publishes internal Ubuntu Docker image#392

Merged
codefromthecrypt merged 2 commits intomasterfrom
docker-polish
Oct 8, 2021
Merged

Publishes internal Ubuntu Docker image#392
codefromthecrypt merged 2 commits intomasterfrom
docker-polish

Conversation

@codefromthecrypt
Copy link
Copy Markdown
Contributor

@codefromthecrypt codefromthecrypt commented Oct 8, 2021

This change...

  • adds an Ubuntu image which can be used to build distributions
  • ensures the image to use runner instead of root like GHA
  • removes go version management in travis
  • removes configure_go.sh

This change...

* adds an Ubuntu image which can be used to build distributions
* ensures the image to use runner instead of root like GHA
* removes go version management in travis
* removes configure_go.sh

Signed-off-by: Adrian Cole <adrian@tetrate.io>
@codefromthecrypt codefromthecrypt marked this pull request as ready for review October 8, 2021 20:36
@codefromthecrypt
Copy link
Copy Markdown
Contributor Author

This change does the necessary to complete packaging tests (#366). I'll redo #387 later to allow seamless transition to go 1.18, but it is lower priority as I don't expect that out soon :D

# ```
jobs:
build-and-push-images:
runs-on: ubuntu-20.04 # Hard-coding an LTS means maintenance, but only once each 2 years!
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

one advantage of switching to explicit versions is now the runner os version matches the docker tag

Comment thread .travis.yml
virt: lxd # faster starting
language: go

go: # Use gimme expression to get latest patch. See https://github.com/travis-ci/gimme
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

so now, the version will be exactly the same GHA vs travis

Signed-off-by: Adrian Cole <adrian@tetrate.io>
@codefromthecrypt codefromthecrypt merged commit bbe328e into master Oct 8, 2021
@codefromthecrypt codefromthecrypt deleted the docker-polish branch October 8, 2021 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant