Skip to content

Conversation

@juliaElastic
Copy link
Contributor

@juliaElastic juliaElastic commented Jun 8, 2023

Summary

The package upload API was made GA, and it had a leftover enterprise license requirement from the time it was only intended for devs. See #79653
This pr removes that requirement.

@juliaElastic juliaElastic added release_note:skip Skip the PR/issue when compiling release notes v8.8.0 v8.9.0 labels Jun 8, 2023
@juliaElastic juliaElastic requested a review from a team as a code owner June 8, 2023 07:23
@juliaElastic juliaElastic self-assigned this Jun 8, 2023
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Jun 8, 2023
@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@ghost
Copy link

ghost commented Jun 8, 2023

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • /oblt-deploy : Deploy a Kibana instance using the Observability test environments.
  • run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)

Copy link
Contributor

@jlind23 jlind23 left a comment

Choose a reason for hiding this comment

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

Thanks, Julia!

@juliaElastic juliaElastic enabled auto-merge (squash) June 8, 2023 07:57
@juliaElastic
Copy link
Contributor Author

@elasticmachine merge upstream

@juliaElastic
Copy link
Contributor Author

@elasticmachine merge upstream

@juliaElastic
Copy link
Contributor Author

@elasticmachine merge upstream

@juliaElastic
Copy link
Contributor Author

@elasticmachine merge upstream

@juliaElastic
Copy link
Contributor Author

The Session View API failure has an open issue that hopefully gets fixed soon: #159275

@juliaElastic
Copy link
Contributor Author

@elasticmachine merge upstream

@juliaElastic juliaElastic merged commit b8619ec into elastic:main Jun 8, 2023
@kibana-ci
Copy link

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Unknown metric groups

ESLint disabled line counts

id before after diff
enterpriseSearch 19 21 +2
securitySolution 414 418 +4
total +6

Total ESLint disabled count

id before after diff
enterpriseSearch 20 22 +2
securitySolution 498 502 +4
total +6

History

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

cc @juliaElastic

@juliaElastic juliaElastic added v8.8.2 and removed v8.8.0 labels Jun 8, 2023
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jun 8, 2023
…astic#159280)

## Summary

The package upload API was made GA, and it had a leftover enterprise
license requirement from the time it was only intended for devs. See
elastic#79653
This pr removes that requirement.

(cherry picked from commit b8619ec)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.8

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

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jun 8, 2023
…nt (#159280) (#159321)

# Backport

This will backport the following commits from `main` to `8.8`:
- [[Fleet] removed package upload API enterprise license requirement
(#159280)](#159280)

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

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

<!--BACKPORT [{"author":{"name":"Julia
Bardi","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-06-08T14:51:54Z","message":"[Fleet]
removed package upload API enterprise license requirement
(#159280)\n\n## Summary\r\n\r\nThe package upload API was made GA, and
it had a leftover enterprise\r\nlicense requirement from the time it was
only intended for devs.
See\r\nhttps://github.com//pull/79653\r\nThis pr removes
that
requirement.","sha":"b8619ec216bebbcfdd27f34c6ef632e5fc2456be","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","v8.8.0","v8.9.0"],"number":159280,"url":"https://github.com/elastic/kibana/pull/159280","mergeCommit":{"message":"[Fleet]
removed package upload API enterprise license requirement
(#159280)\n\n## Summary\r\n\r\nThe package upload API was made GA, and
it had a leftover enterprise\r\nlicense requirement from the time it was
only intended for devs.
See\r\nhttps://github.com//pull/79653\r\nThis pr removes
that
requirement.","sha":"b8619ec216bebbcfdd27f34c6ef632e5fc2456be"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"8.8","label":"v8.8.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/159280","number":159280,"mergeCommit":{"message":"[Fleet]
removed package upload API enterprise license requirement
(#159280)\n\n## Summary\r\n\r\nThe package upload API was made GA, and
it had a leftover enterprise\r\nlicense requirement from the time it was
only intended for devs.
See\r\nhttps://github.com//pull/79653\r\nThis pr removes
that requirement.","sha":"b8619ec216bebbcfdd27f34c6ef632e5fc2456be"}}]}]
BACKPORT-->

Co-authored-by: Julia Bardi <[email protected]>
@mistic
Copy link
Member

mistic commented Jun 8, 2023

This PR backport didn't make it into the latest BC for 8.8.1. Updating the labels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.8.2 v8.9.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants