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] Fix config argument order #151052

Merged
merged 1 commit into from
Feb 13, 2023
Merged

Conversation

jbudz
Copy link
Member

@jbudz jbudz commented Feb 13, 2023

Snapshot builds are currently being marked as release builds in the config service. This impacts the release property in package.json snapshot builds and pull request deployments.

This was introduced in 1b85815#diff-dbea24da2a777429d025c1da037dd966d65bff2c97a7b78b82a33532f3ad06d9R63

Snapshot builds are currently being marked as release builds in the
config service.  This impacts the release property in package.json
snapshot builds and pull request deployments.
@jbudz jbudz added Team:Operations Team label for Operations Team release_note:skip Skip the PR/issue when compiling release notes ci:cloud-deploy Create or update a Cloud deployment labels Feb 13, 2023
@jbudz jbudz requested a review from a team as a code owner February 13, 2023 18:11
@elasticmachine
Copy link
Contributor

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

@kibana-ci
Copy link
Collaborator

kibana-ci commented Feb 13, 2023

💔 Build Failed

Failed CI Steps

Metrics [docs]

✅ unchanged

History

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

@jbudz
Copy link
Member Author

jbudz commented Feb 13, 2023

The cloud deployment failure above is expected for now. We are creating the correct artifacts with more unit tests added - merging away.

@jbudz jbudz merged commit 17855ba into elastic:main Feb 13, 2023
@kibanamachine kibanamachine added v8.8.0 backport:skip This commit does not require backporting labels Feb 13, 2023
mistic pushed a commit to mistic/kibana that referenced this pull request May 17, 2023
Snapshot builds are currently being marked as release builds in the
config service. This impacts the release property in package.json
snapshot builds and pull request deployments.

This was introduced in
elastic@1b85815#diff-dbea24da2a777429d025c1da037dd966d65bff2c97a7b78b82a33532f3ad06d9R63

(cherry picked from commit 17855ba)
@mistic
Copy link
Member

mistic commented May 17, 2023

💚 All backports created successfully

Status Branch Result
8.7

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 May 17, 2023
# Backport

This will backport the following commits from `main` to `8.7`:
- [[build] Fix config argument order
(#151052)](#151052)

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

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

<!--BACKPORT
[{"author":{"name":"Jon","email":"jon@elastic.co"},"sourceCommit":{"committedDate":"2023-02-13T21:48:28Z","message":"[build]
Fix config argument order (#151052)\n\nSnapshot builds are currently
being marked as release builds in the\r\nconfig service. This impacts
the release property in package.json\r\nsnapshot builds and pull request
deployments.\r\n\r\nThis was introduced
in\r\nhttps://github.com/elastic/kibana/commit/1b8581540295fde746dae6b4a09d74fb5821bfef#diff-dbea24da2a777429d025c1da037dd966d65bff2c97a7b78b82a33532f3ad06d9R63","sha":"17855ba5a240ce2cd60f318b49f0040d0cfabbbd","branchLabelMapping":{"^v8.8.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:Operations","release_note:skip","backport:skip","ci:cloud-deploy","v8.8.0"],"number":151052,"url":"#151052
Fix config argument order (#151052)\n\nSnapshot builds are currently
being marked as release builds in the\r\nconfig service. This impacts
the release property in package.json\r\nsnapshot builds and pull request
deployments.\r\n\r\nThis was introduced
in\r\nhttps://github.com/elastic/kibana/commit/1b8581540295fde746dae6b4a09d74fb5821bfef#diff-dbea24da2a777429d025c1da037dd966d65bff2c97a7b78b82a33532f3ad06d9R63","sha":"17855ba5a240ce2cd60f318b49f0040d0cfabbbd"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.8.0","labelRegex":"^v8.8.0$","isSourceBranch":true,"state":"MERGED","url":"#151052
Fix config argument order (#151052)\n\nSnapshot builds are currently
being marked as release builds in the\r\nconfig service. This impacts
the release property in package.json\r\nsnapshot builds and pull request
deployments.\r\n\r\nThis was introduced
in\r\nhttps://github.com/elastic/kibana/commit/1b8581540295fde746dae6b4a09d74fb5821bfef#diff-dbea24da2a777429d025c1da037dd966d65bff2c97a7b78b82a33532f3ad06d9R63","sha":"17855ba5a240ce2cd60f318b49f0040d0cfabbbd"}}]}]
BACKPORT-->

Co-authored-by: Jon <jon@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:skip This commit does not require backporting ci:cloud-deploy Create or update a Cloud deployment release_note:skip Skip the PR/issue when compiling release notes Team:Operations Team label for Operations Team v8.7.2 v8.8.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants