Skip to content
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] Add cloud docker images #107949

Merged
merged 23 commits into from
Sep 10, 2021
Merged

[build] Add cloud docker images #107949

merged 23 commits into from
Sep 10, 2021

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Aug 9, 2021

This adds a new build task to create cloud docker images. These images
include metricbeat and filebeat. Archives for both are expected to
be in the target folder prior to starting the build.

The cloud image can be generated with node scripts/build --all-platforms --docker-cloud

Closes #99916

This adds a new build task to create cloud docker images.  These images
include metricbeat and filebeat.  Archives are for both are expected to
be in the target folder prior to starting the build.

The cloud image can be generated with `node scripts/build
--all-platforms --docker-cloud`

Closes elastic#99916
@jbudz jbudz added Team:Operations Team label for Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes labels Aug 9, 2021
@jbudz jbudz marked this pull request as ready for review August 10, 2021 19:43
@jbudz jbudz requested a review from a team as a code owner August 10, 2021 19:43
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@jbudz
Copy link
Member Author

jbudz commented Aug 11, 2021

@elasticmachine merge upstream

@jbudz
Copy link
Member Author

jbudz commented Aug 13, 2021

@elasticmachine merge upstream

@jbudz
Copy link
Member Author

jbudz commented Aug 16, 2021

@elasticmachine merge upstream

@jbudz
Copy link
Member Author

jbudz commented Aug 16, 2021

cc @mieciu

Copy link
Contributor

@mieciu mieciu left a comment

Choose a reason for hiding this comment

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

The entrypoint is fine, but we'd need to change the default command to
CMD ["/app/kibana.sh"]

Copy link
Contributor

@mieciu mieciu left a comment

Choose a reason for hiding this comment

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

CMD needs to be set to /app/kibana.sh. See how Elasticsearch got away with this.

@jbudz jbudz added the v7.16.0 label Aug 23, 2021
@jbudz
Copy link
Member Author

jbudz commented Aug 23, 2021

~/dev/kibana/target(build/cloud-testing*) » docker load --input kibana-cloud-8.0.0-SNAPSHOT-docker-image.tar.gz
Loaded image: docker.elastic.co/kibana/kibana-cloud:8.0.0-SNAPSHOT
~/dev/kibana/target(build/cloud-testing*) » docker run -it docker.elastic.co/kibana/kibana-cloud:8.0.0-SNAPSHOT /bin/bash   
bash-4.4$ ls -lah /opt
total 16K
drwxr-xr-x 1 root root 4.0K Aug 23 18:40 .
drwxr-xr-x 1 root root 4.0K Aug 23 18:44 ..
drwxr-xr-x 5 root root 4.0K Aug 23 18:39 filebeat
lrwxrwxrwx 1 root root   17 Aug 23 18:40 kibana -> /usr/share/kibana
drwxr-xr-x 5 root root 4.0K Aug 23 18:39 metricbeat
bash-4.4$ cat /app/kibana.sh 
#!/bin/sh
exec /usr/local/bin/kibana-docker

@jbudz
Copy link
Member Author

jbudz commented Aug 30, 2021

@elasticmachine merge upstream

@jbudz
Copy link
Member Author

jbudz commented Sep 7, 2021

I pushed a small change to move the beats artifacts outside of target. The target folder is managed by the build process and cleans out artifacts prior to producing new builds.

@jbudz
Copy link
Member Author

jbudz commented Sep 9, 2021

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@jbudz jbudz merged commit 163fa7d into elastic:master Sep 10, 2021
@jbudz jbudz deleted the build/cloud-testing branch September 10, 2021 01:40
@jbudz jbudz added the auto-backport Deprecated - use backport:version if exact versions are needed label Sep 10, 2021
kibanamachine added a commit to kibanamachine/kibana that referenced this pull request Sep 10, 2021
Co-authored-by: Przemysław Hejman <[email protected]>
Co-authored-by: Kibana Machine <[email protected]>
@kibanamachine
Copy link
Contributor

💚 Backport successful

Status Branch Result
7.x

This backport PR will be merged automatically after passing CI.

kibanamachine added a commit that referenced this pull request Sep 10, 2021
Co-authored-by: Przemysław Hejman <[email protected]>
Co-authored-by: Kibana Machine <[email protected]>

Co-authored-by: Jonathan Budzenski <[email protected]>
Co-authored-by: Przemysław Hejman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.16.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[build] Produce Cloud container as part of release manager
5 participants