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

chore(NA): support bazel and kbn packages in parallel on kbn pm and on distributable build scripts #89961

Merged
merged 25 commits into from
Feb 4, 2021

Conversation

mistic
Copy link
Member

@mistic mistic commented Feb 2, 2021

One step forward on #69706

That PR enable us of actually starting developing the BUILD files for Bazel packages.
It enables an auto detector on kbn-pm that will classify packages as Bazel packages. It also introduces a couple of useful methods on kbn-pm so we can distinguish between bazelProjectsOnly and nonBazelProjects only.
Finally it changes both the kbn bootstrap and our distributable build scripts so we can have at the same time kbn and Bazel packages being built and consumed.

@mistic mistic added chore Team:Operations Team label for Operations Team build v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.12.0 labels Feb 2, 2021
@mistic mistic requested a review from tylersmalley February 2, 2021 01:58
@mistic mistic requested a review from a team as a code owner February 2, 2021 01:58
@elasticmachine
Copy link
Contributor

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

@mistic

This comment has been minimized.

@mistic

This comment has been minimized.

@mistic mistic force-pushed the support-bazel-packages-on-kbn-pm branch from 1233850 to 6c4363c Compare February 3, 2021 13:23
@mistic

This comment has been minimized.

@tylersmalley
Copy link
Contributor

LGTM: verified by converting elastic-datemath: https://github.com/mistic/kibana/compare/support-bazel-packages-on-kbn-pm...tylersmalley:testing/support-bazel-packages-on-kbn-pm?expand=1

 info [bazel] Starting local Bazel server and connecting to it...
 info [bazel] INFO: Invocation ID: da8be204-dbb1-4860-a981-f41e68d28fbf
 info [bazel] INFO: Streaming build results to: https://app.buildbuddy.io/invocation/da8be204-dbb1-4860-a981-f41e68d28fbf
 info [bazel] INFO: Analyzed target //packages:build (8 packages loaded, 547 targets configured).
 info [bazel] INFO: Found 1 target...
 info [bazel] INFO: Elapsed time: 4.541s, Critical Path: 0.03s
 info [bazel] INFO: 1 process: 1 internal.
 info [bazel] INFO: Streaming build results to: https://app.buildbuddy.io/invocation/da8be204-dbb1-4860-a981-f41e68d28fbf
 info [bazel] 
 succ 28 bootstrap builds are cached

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@mistic mistic merged commit 23d5ffb into elastic:master Feb 4, 2021
@mistic
Copy link
Member Author

mistic commented Feb 4, 2021

7.x/7.12: 80b018e

mistic added a commit to mistic/kibana that referenced this pull request Feb 4, 2021
…n distributable build scripts (elastic#89961)

* chore(NA): bazel projects support initial flag on kbn pm

* chore(NA): every needed step to build production bazel packages except the final function

* chore(NA): include build bazel production projects on kibana distributable build

* chore(NA): support bazel packages when creating the package.json

* chore(NA): including last changes on kbn pm and build distributable to support both bazel and kbn packages

* chore(NA): missing annotation on build bazel packages

* chore(NA): changed values on bazelrc common

* chore(NA): fix bazel common rc

* chore(NA): auto discovery if a kbn package is a Bazel package

* chore(NA): last details to make bazel packages to built on distributable scripts

* chore(NA): removed wrongly added line to x-pack package.json

* chore(NA): apply correct formating

* chore(NA): move into bazel bin

* chore(NA): merge chnages on kbn pm

* chore(NA): correctly setup ignore files for new bazel aggregated folder

* chore(NA): merge with last master

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
tylersmalley pushed a commit that referenced this pull request Feb 4, 2021
…n distributable build scripts (#89961) (#90256)

* chore(NA): bazel projects support initial flag on kbn pm

* chore(NA): every needed step to build production bazel packages except the final function

* chore(NA): include build bazel production projects on kibana distributable build

* chore(NA): support bazel packages when creating the package.json

* chore(NA): including last changes on kbn pm and build distributable to support both bazel and kbn packages

* chore(NA): missing annotation on build bazel packages

* chore(NA): changed values on bazelrc common

* chore(NA): fix bazel common rc

* chore(NA): auto discovery if a kbn package is a Bazel package

* chore(NA): last details to make bazel packages to built on distributable scripts

* chore(NA): removed wrongly added line to x-pack package.json

* chore(NA): apply correct formating

* chore(NA): move into bazel bin

* chore(NA): merge chnages on kbn pm

* chore(NA): correctly setup ignore files for new bazel aggregated folder

* chore(NA): merge with last master

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build chore release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.12.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants