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

fix(NA): yarn env vars for node_modules mirrors #163549

Merged

Conversation

mistic
Copy link
Member

@mistic mistic commented Aug 10, 2023

This PR fixes the setup we have for the node_module mirrors vars that are overriding and pointing into our middle cache. The previous configuration was not working as intended as the env vars set globally on CI never ended up in the bazel managed yarn install.

@mistic mistic added chore Team:Operations Team label for Operations Team release_note:skip Skip the PR/issue when compiling release notes backport:all-open Backport to all branches that could still receive a release v8.10.0 labels Aug 10, 2023
@mistic mistic marked this pull request as ready for review August 10, 2023 00:56
@mistic mistic requested a review from a team as a code owner August 10, 2023 00:56
@elasticmachine
Copy link
Contributor

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

@mistic mistic enabled auto-merge (squash) August 10, 2023 02:15
@kibana-ci
Copy link
Collaborator

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

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

@mistic mistic merged commit 17936ff into elastic:main Aug 10, 2023
21 checks passed
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Aug 10, 2023
This PR fixes the setup we have for the node_module mirrors vars that
are overriding and pointing into our middle cache. The previous
configuration was not working as intended as the env vars set globally
on CI never ended up in the bazel managed yarn install.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 17936ff)
@kibanamachine
Copy link
Contributor

💔 Some backports could not be created

Status Branch Result
7.17 Backport failed because of merge conflicts

You might need to backport the following PRs to 7.17:
- [Ops] Bump Node.js to version 18 (#160289)
- Bump Node.js to 16.16.0 (#135926)
8.9

Note: Successful backport PRs will be merged automatically after passing CI.

Manual backport

To create the backport manually run:

node scripts/backport --pr 163549

Questions ?

Please refer to the Backport tool documentation

jloleysens added a commit to jloleysens/kibana that referenced this pull request Aug 10, 2023
* main: (108 commits)
  [Telemetry Schema Validation] Allow `null` on `string` (elastic#163499)
  [Search] Add Slack and Gmail connectors (elastic#163321)
  [ML] Provide hints for empty fields in dropdown options in Anomaly detection & Transform creation wizards, Change point detection view (elastic#163371)
  chore(slo): Add response required fields (elastic#163430)
  [AO] Fix add_to_case functional test (elastic#163155)
  unskip license type functional test (elastic#163199)
  fix(NA): yarn env vars for node_modules mirrors (elastic#163549)
  [Response Ops][Task Manager] Expose SLI metrics in HTTP API (elastic#162178)
  [Logs UI] Adapt test to ES highlighting changes and unskip (elastic#163592)
  [Infra UI] Implement Telemetry on 'Show' buttons within Inventory (elastic#163587)
  [Enterprise Search]Migrate all usages of EuiPage*_Deprecated (elastic#163482)
  fix(slo): settings and access for serverless (elastic#163514)
  [Infra UI] Implement telemetry for the asset details flyout (elastic#163078)
  [Fleet] Add a banner to the top of the Kafka Output UI to say that Elastic Defend integration is not supported (elastic#163579)
  [Fleet] Re-enable and fix Fleet policy secret integration tests (elastic#163428)
  [Fleet] add managed to imported saved object (elastic#163526)
  [Index Management] Disable index actions using contextRef (elastic#163475)
  [Discover] Inline shard failures warnings (elastic#161271)
  [Security Solution][Detection engine] skips geo_point non-ecs validation (elastic#163487)
  Update EUI layout components in bfetch example plugin (elastic#163490)
  ...
kibanamachine added a commit that referenced this pull request Aug 10, 2023
)

# Backport

This will backport the following commits from `main` to `8.9`:
- [fix(NA): yarn env vars for node_modules mirrors
(#163549)](#163549)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Tiago
Costa","email":"tiago.costa@elastic.co"},"sourceCommit":{"committedDate":"2023-08-10T14:35:34Z","message":"fix(NA):
yarn env vars for node_modules mirrors (#163549)\n\nThis PR fixes the
setup we have for the node_module mirrors vars that\r\nare overriding
and pointing into our middle cache. The previous\r\nconfiguration was
not working as intended as the env vars set globally\r\non CI never
ended up in the bazel managed yarn install.\r\n\r\nCo-authored-by:
Kibana Machine
<42973632+kibanamachine@users.noreply.github.com>","sha":"17936ffd21d4b4b274d2cda90902764ed0d4ae07","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["chore","Team:Operations","release_note:skip","backport:all-open","v8.10.0"],"number":163549,"url":"#163549:
yarn env vars for node_modules mirrors (#163549)\n\nThis PR fixes the
setup we have for the node_module mirrors vars that\r\nare overriding
and pointing into our middle cache. The previous\r\nconfiguration was
not working as intended as the env vars set globally\r\non CI never
ended up in the bazel managed yarn install.\r\n\r\nCo-authored-by:
Kibana Machine
<42973632+kibanamachine@users.noreply.github.com>","sha":"17936ffd21d4b4b274d2cda90902764ed0d4ae07"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"#163549:
yarn env vars for node_modules mirrors (#163549)\n\nThis PR fixes the
setup we have for the node_module mirrors vars that\r\nare overriding
and pointing into our middle cache. The previous\r\nconfiguration was
not working as intended as the env vars set globally\r\non CI never
ended up in the bazel managed yarn install.\r\n\r\nCo-authored-by:
Kibana Machine
<42973632+kibanamachine@users.noreply.github.com>","sha":"17936ffd21d4b4b274d2cda90902764ed0d4ae07"}}]}]
BACKPORT-->

Co-authored-by: Tiago Costa <tiago.costa@elastic.co>
mistic added a commit to mistic/kibana that referenced this pull request Aug 15, 2023
This PR fixes the setup we have for the node_module mirrors vars that
are overriding and pointing into our middle cache. The previous
configuration was not working as intended as the env vars set globally
on CI never ended up in the bazel managed yarn install.

Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>
(cherry picked from commit 17936ff)

# Conflicts:
#	WORKSPACE.bazel
@mistic
Copy link
Member Author

mistic commented Aug 15, 2023

💚 All backports created successfully

Status Branch Result
7.17

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

mistic added a commit that referenced this pull request Aug 15, 2023
…3897)

# Backport

This will backport the following commits from `main` to `7.17`:
- [fix(NA): yarn env vars for node_modules mirrors
(#163549)](#163549)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Tiago
Costa","email":"tiago.costa@elastic.co"},"sourceCommit":{"committedDate":"2023-08-10T14:35:34Z","message":"fix(NA):
yarn env vars for node_modules mirrors (#163549)\n\nThis PR fixes the
setup we have for the node_module mirrors vars that\r\nare overriding
and pointing into our middle cache. The previous\r\nconfiguration was
not working as intended as the env vars set globally\r\non CI never
ended up in the bazel managed yarn install.\r\n\r\nCo-authored-by:
Kibana Machine
<42973632+kibanamachine@users.noreply.github.com>","sha":"17936ffd21d4b4b274d2cda90902764ed0d4ae07","branchLabelMapping":{"^v8.10.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["chore","Team:Operations","release_note:skip","backport:all-open","v8.10.0","v8.9.1"],"number":163549,"url":"#163549:
yarn env vars for node_modules mirrors (#163549)\n\nThis PR fixes the
setup we have for the node_module mirrors vars that\r\nare overriding
and pointing into our middle cache. The previous\r\nconfiguration was
not working as intended as the env vars set globally\r\non CI never
ended up in the bazel managed yarn install.\r\n\r\nCo-authored-by:
Kibana Machine
<42973632+kibanamachine@users.noreply.github.com>","sha":"17936ffd21d4b4b274d2cda90902764ed0d4ae07"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.10.0","labelRegex":"^v8.10.0$","isSourceBranch":true,"state":"MERGED","url":"#163549:
yarn env vars for node_modules mirrors (#163549)\n\nThis PR fixes the
setup we have for the node_module mirrors vars that\r\nare overriding
and pointing into our middle cache. The previous\r\nconfiguration was
not working as intended as the env vars set globally\r\non CI never
ended up in the bazel managed yarn install.\r\n\r\nCo-authored-by:
Kibana Machine
<42973632+kibanamachine@users.noreply.github.com>","sha":"17936ffd21d4b4b274d2cda90902764ed0d4ae07"}},{"branch":"8.9","label":"v8.9.1","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"url":"#163617
fix(NA): yarn env vars for node_modules mirrors (#163549) (#163617)\n\n#
Backport\n\nThis will backport the following commits from `main` to
`8.9`:\n- [fix(NA): yarn env vars for node_modules
mirrors\n(#163549)](https://github.com/elastic/kibana/pull/163549)\n\n<!---
Backport version: 8.9.7 -->\n\n### Questions ?\nPlease refer to the
[Backport
tool\ndocumentation](https://github.com/sqren/backport)\n\n<!--BACKPORT
[{\"author\":{\"name\":\"Tiago\nCosta\",\"email\":\"tiago.costa@elastic.co\"},\"sourceCommit\":{\"committedDate\":\"2023-08-10T14:35:34Z\",\"message\":\"fix(NA):\nyarn
env vars for node_modules mirrors (#163549)\\n\\nThis PR fixes
the\nsetup we have for the node_module mirrors vars that\\r\\nare
overriding\nand pointing into our middle cache. The
previous\\r\\nconfiguration was\nnot working as intended as the env vars
set globally\\r\\non CI never\nended up in the bazel managed yarn
install.\\r\\n\\r\\nCo-authored-by:\nKibana
Machine\n<42973632+kibanamachine@users.noreply.github.com>\",\"sha\":\"17936ffd21d4b4b274d2cda90902764ed0d4ae07\",\"branchLabelMapping\":{\"^v8.10.0$\":\"main\",\"^v(\\\\d+).(\\\\d+).\\\\d+$\":\"$1.$2\"}},\"sourcePullRequest\":{\"labels\":[\"chore\",\"Team:Operations\",\"release_note:skip\",\"backport:all-open\",\"v8.10.0\"],\"number\":163549,\"url\":\"#163549
env vars for node_modules mirrors (#163549)\\n\\nThis PR fixes
the\nsetup we have for the node_module mirrors vars that\\r\\nare
overriding\nand pointing into our middle cache. The
previous\\r\\nconfiguration was\nnot working as intended as the env vars
set globally\\r\\non CI never\nended up in the bazel managed yarn
install.\\r\\n\\r\\nCo-authored-by:\nKibana
Machine\n<42973632+kibanamachine@users.noreply.github.com>\",\"sha\":\"17936ffd21d4b4b274d2cda90902764ed0d4ae07\"}},\"sourceBranch\":\"main\",\"suggestedTargetBranches\":[],\"targetPullRequestStates\":[{\"branch\":\"main\",\"label\":\"v8.10.0\",\"labelRegex\":\"^v8.10.0$\",\"isSourceBranch\":true,\"state\":\"MERGED\",\"url\":\"#163549
env vars for node_modules mirrors (#163549)\\n\\nThis PR fixes
the\nsetup we have for the node_module mirrors vars that\\r\\nare
overriding\nand pointing into our middle cache. The
previous\\r\\nconfiguration was\nnot working as intended as the env vars
set globally\\r\\non CI never\nended up in the bazel managed yarn
install.\\r\\n\\r\\nCo-authored-by:\nKibana
Machine\n<42973632+kibanamachine@users.noreply.github.com>\",\"sha\":\"17936ffd21d4b4b274d2cda90902764ed0d4ae07\"}}]}]\nBACKPORT-->\n\nCo-authored-by:
Tiago Costa <tiago.costa@elastic.co>"}}]}] BACKPORT-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:all-open Backport to all branches that could still receive a release chore release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v7.17.13 v8.9.1 v8.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants