Skip to content

Conversation

@zliang-akamai
Copy link
Member

@zliang-akamai zliang-akamai commented Jun 27, 2023

📝 Description

The based image of the docker files are outdated. And some older docker files are no longer used. We should cleanup the unused ones and upgrade the useful one(s).

✔️ How to Test

  1. Setup GITHUB_TOKEN with the basic read public repo scope and LINODE_TOEKN in the env.
  2. docker build --build-arg="github_token=$GITHUB_TOKEN" .
  3. docker images, then copy the most recent image ID built by step 2.
  4. docker run --rm -it -e LINODE_CLI_TOKEN=$LINODE_TOKEN the_image_id linodes list

@zliang-akamai zliang-akamai force-pushed the zhiwei/release-cleanup-and-upgrade branch from f3ecc5c to 9d198c8 Compare June 27, 2023 19:30
@zliang-akamai zliang-akamai changed the title Zhiwei/release cleanup and upgrade Dockerfiles upgrade and cleanup Jun 27, 2023
@zliang-akamai zliang-akamai marked this pull request as ready for review June 27, 2023 19:31
@zliang-akamai zliang-akamai marked this pull request as draft June 27, 2023 19:33
@zliang-akamai zliang-akamai marked this pull request as ready for review June 27, 2023 20:24
@zliang-akamai zliang-akamai requested a review from a team June 27, 2023 20:24
Co-authored-by: Lena Garber <114949949+lgarber-akamai@users.noreply.github.com>
@zliang-akamai zliang-akamai requested review from a team and lgarber-akamai June 28, 2023 16:38
Copy link
Contributor

@lgarber-akamai lgarber-akamai left a comment

Choose a reason for hiding this comment

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

Image is down to 285MB, nice work!

@zliang-akamai zliang-akamai requested a review from a team July 5, 2023 05:49
Copy link
Contributor

@yec-akamai yec-akamai left a comment

Choose a reason for hiding this comment

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

Nice work!

@zliang-akamai zliang-akamai changed the title Dockerfiles upgrade and cleanup Dockerfiles upgrade and BATS test container and script cleanup Jul 10, 2023
@zliang-akamai zliang-akamai changed the title Dockerfiles upgrade and BATS test container and script cleanup Dockerfile upgrade and BATS test related Dockerfiles and scripts cleanup Jul 10, 2023
@zliang-akamai zliang-akamai merged commit f1a64ae into linode:dev Jul 10, 2023
@zliang-akamai zliang-akamai deleted the zhiwei/release-cleanup-and-upgrade branch July 10, 2023 06:13
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.

3 participants