Skip to content

chore(NA): introduce new yarn kbn reset command to support bazel workflow#89597

Merged
mistic merged 34 commits intoelastic:masterfrom
mistic:add-kbn-destroy-cmd-for-bazel-integration
Feb 3, 2021
Merged

chore(NA): introduce new yarn kbn reset command to support bazel workflow#89597
mistic merged 34 commits intoelastic:masterfrom
mistic:add-kbn-destroy-cmd-for-bazel-integration

Conversation

@mistic
Copy link
Copy Markdown
Contributor

@mistic mistic commented Jan 28, 2021

One step forward on #69706

On Bazel we have two kind of cleans: a soft one and an harder one.
When migrating into the Bazel new workflow we will have to manage the swaps between branches both using Bazel but also the swaps between Branches where one of them could not be using Bazel.

The new yarn kbn reset should be the way to go when switching from a non Bazel branch into a Bazel one and the yarn kbn clean the 99% of the time command to use when switching or working only with Bazel branches.

@mistic mistic added chore Team:Operations Kibana-Operations Team build v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.12.0 labels Jan 28, 2021
@mistic mistic requested a review from tylersmalley January 28, 2021 17:17
@mistic mistic requested a review from a team as a code owner January 28, 2021 17:17
@elasticmachine
Copy link
Copy Markdown
Contributor

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

@mistic

This comment has been minimized.

@mistic

This comment has been minimized.

@mistic

This comment has been minimized.

@mistic

This comment has been minimized.

@mistic

This comment has been minimized.

@mistic

This comment has been minimized.

@mistic

This comment has been minimized.

Copy link
Copy Markdown
Member

@tylersmalley tylersmalley left a comment

Choose a reason for hiding this comment

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

LGTM

@kibanamachine
Copy link
Copy Markdown
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 a0d5d56 into elastic:master Feb 3, 2021
@mistic
Copy link
Copy Markdown
Contributor Author

mistic commented Feb 3, 2021

7.x/7.12: 1c6adb4

mistic added a commit to mistic/kibana that referenced this pull request Feb 3, 2021
…flow (elastic#89597)

* chore(NA): introduce new yarn kbn destroy command to support bazel workflow

* chore(NA): update wording for new kbn pm changes

* chore(NA): update .bazelrc.common

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update .bazelrc

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update packages/kbn-pm/src/commands/clean.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update packages/kbn-pm/src/commands/destroy.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update packages/kbn-pm/src/commands/clean.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update packages/kbn-pm/src/commands/destroy.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): rename destroy command into reset

* chore(NA): update packages/kbn-pm/src/commands/clean.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): restore old behaviour on kbn clean

* chore(NA): update reset command to delete bazel caches on disk

* chore(NA): update packages/kbn-pm/src/commands/clean.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update prefix to match bazel defined one for cache settings

* chore(NA): update kbn pm dist file

* chore(NA): update kbn pm dist

* chore(NA): update gitignore

* chore(NA): add new ignore files after changed bazel aggregated folder

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Tyler Smalley <tylersmalley@me.com>
# Conflicts:
#	packages/kbn-pm/dist/index.js
mistic added a commit that referenced this pull request Feb 4, 2021
…l workflow (#89597) (#90244)

* chore(NA): introduce new yarn kbn reset command to support bazel workflow (#89597)

* chore(NA): introduce new yarn kbn destroy command to support bazel workflow

* chore(NA): update wording for new kbn pm changes

* chore(NA): update .bazelrc.common

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update .bazelrc

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update packages/kbn-pm/src/commands/clean.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update packages/kbn-pm/src/commands/destroy.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update packages/kbn-pm/src/commands/clean.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update packages/kbn-pm/src/commands/destroy.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): rename destroy command into reset

* chore(NA): update packages/kbn-pm/src/commands/clean.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): restore old behaviour on kbn clean

* chore(NA): update reset command to delete bazel caches on disk

* chore(NA): update packages/kbn-pm/src/commands/clean.ts

Co-authored-by: Tyler Smalley <tylersmalley@me.com>

* chore(NA): update prefix to match bazel defined one for cache settings

* chore(NA): update kbn pm dist file

* chore(NA): update kbn pm dist

* chore(NA): update gitignore

* chore(NA): add new ignore files after changed bazel aggregated folder

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
Co-authored-by: Tyler Smalley <tylersmalley@me.com>
# Conflicts:
#	packages/kbn-pm/dist/index.js

* chore(NA): update changes on kbm pm

* chore(NA): fix jest test
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 Kibana-Operations Team v7.12.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants