-
Notifications
You must be signed in to change notification settings - Fork 8.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[build/docker] Upgrade Ubuntu base image to 22.04 #162282
Conversation
buildkite test this |
Pinging @elastic/kibana-operations (Team:Operations) |
@@ -189,7 +189,7 @@ ENTRYPOINT ["/bin/tini", "--"] | |||
CMD ["/app/kibana.sh"] | |||
# Generate a stub command that will be overwritten at runtime | |||
RUN mkdir /app && \ | |||
echo -e '#!/bin/bash\nexec /usr/local/bin/kibana-docker' > /app/kibana.sh && \ | |||
/usr/bin/echo -e '#!/bin/bash\nexec /usr/local/bin/kibana-docker' > /app/kibana.sh && \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is for local testing and can be ignored, it's meant to be overwritten. RUN
uses sh, which has a builtin echo that doesn't support the -e
flag.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for upgrading Jon. The deployment images look fine. Think we need a doc update here:
kibana/docs/setup/docker.asciidoc
Line 13 in 8fd827f
base image is https://hub.docker.com/_/ubuntu[ubuntu:20.04]. |
Also, do you think this could be considered a breaking change for downstream users? Do users use the Kibana image as a base image?
💚 Build Succeeded
Metrics [docs]Canvas Sharable Runtime
History
To update your PR or re-run it, just comment with: |
I'm going to manually open a 7.17 backport, splitting off the docs changes to a separate pull request that's merged after the next 7.17 release. Will link in a moment. |
Release note: The default container's base image has been upgraded from Ubuntu 20.04 to 22.04 (cherry picked from commit 9d139f6)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
Release note: The default container's base image has been upgraded from Ubuntu 20.04 to 22.04
…169580) # Backport This will backport the following commits from `main` to `8.11`: - [[build/docker] Upgrade Ubuntu base image to 22.04 (#162282)](#162282) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Jon","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-10-23T19:49:06Z","message":"[build/docker] Upgrade Ubuntu base image to 22.04 (#162282)\n\nRelease note:\r\nThe default container's base image has been upgraded from Ubuntu 20.04\r\nto 22.04","sha":"9d139f6192647a91010e95c84d651644158abc69","branchLabelMapping":{"^v8.12.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:enhancement","Team:Operations","backport:prev-minor","ci:cloud-redeploy","ci:build-serverless-image","v8.12.0"],"number":162282,"url":"https://github.com/elastic/kibana/pull/162282","mergeCommit":{"message":"[build/docker] Upgrade Ubuntu base image to 22.04 (#162282)\n\nRelease note:\r\nThe default container's base image has been upgraded from Ubuntu 20.04\r\nto 22.04","sha":"9d139f6192647a91010e95c84d651644158abc69"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.12.0","labelRegex":"^v8.12.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/162282","number":162282,"mergeCommit":{"message":"[build/docker] Upgrade Ubuntu base image to 22.04 (#162282)\n\nRelease note:\r\nThe default container's base image has been upgraded from Ubuntu 20.04\r\nto 22.04","sha":"9d139f6192647a91010e95c84d651644158abc69"}}]}] BACKPORT--> Co-authored-by: Jon <[email protected]>
…stic#162282) (elastic#169580)" This reverts commit a382569.
…stic#162282) (elastic#169581)" This reverts commit c8260c3.
There's issues starting node on hosts running older versions of Ubuntu.
|
…2282)" (elastic#170430) This reverts commit 9d139f6.
Release note:
reverted
The default container's base image has been upgraded from Ubuntu 20.04 to 22.04