Skip to content

Conversation

@Yikun
Copy link
Member

@Yikun Yikun commented Aug 4, 2022

What changes were proposed in this pull request?

Change infra image to static tag: ubuntu:focal-20220801 (current latest 20.04). This patch don't bring any real change on image because ubuntu:20.04 and ubuntu:focal-20220801 are point to same hash at current time.

$ docker run -ti ubuntu:focal-20220801 bash
Digest: sha256:af5efa9c28de78b754777af9b4d850112cad01899a5d37d2617bb94dc63a49aa
Status: Downloaded newer image for ubuntu:focal-20220801

$ docker run -ti ubuntu:20.04 bash
Digest: sha256:af5efa9c28de78b754777af9b4d850112cad01899a5d37d2617bb94dc63a49aa
Status: Downloaded newer image for ubuntu:20.04

Why are the changes needed?

  • The ubuntu:20.04 is not a static tag, it is for ubuntu latest 20.04 version, Ubuntu upgrades the version every 1-2 months.
  • If we don't pin the ubuntu version to specific static tag, once the ubuntu upgrade the version, the image cache will be outdate. Then the infra image of CI job will do full refresh, it makes CI unstable just like SPARK-39831 .

So, we need a planned upgrade of Ubuntu in infra to aovid unstable.

Does this PR introduce any user-facing change?

No

How was this patch tested?

CI passed

@github-actions github-actions bot added the BUILD label Aug 4, 2022
@Yikun Yikun changed the title [WIP] Change infra image to static hash ubuntu:focal-20220801 [SPARK-39980][INFRA] Change infra image to static hash ubuntu:focal-20220801 Aug 4, 2022
@Yikun Yikun changed the title [SPARK-39980][INFRA] Change infra image to static hash ubuntu:focal-20220801 [SPARK-39980][INFRA] Change infra image to static tag: ubuntu:focal-20220801 Aug 4, 2022
@Yikun Yikun marked this pull request as ready for review August 4, 2022 13:04
@Yikun
Copy link
Member Author

Yikun commented Aug 5, 2022

@HyukjinKwon Ready to go.

@HyukjinKwon
Copy link
Member

Merged to master.

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.

2 participants