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

Add a new role for APM Server 8.7.0+ #6605

Merged
merged 1 commit into from
Mar 28, 2023
Merged

Conversation

axw
Copy link
Member

@axw axw commented Mar 28, 2023

Add a new role for APM Server 8.7.0+, adding read privileges for the system indices .apm-agent-configuration and .apm-source-map.

Add a new role for APM Server 8.7.0+, adding
read privileges for the system indices
.apm-agent-configuration and .apm-source-map.
@botelastic botelastic bot added the triage label Mar 28, 2023
@pebrc
Copy link
Collaborator

pebrc commented Mar 28, 2023

@thbkrkr this probably still needs to go into 2.7!
@axw will APM server 8.7 work without this i.e. is this a blocker for ECK + APM server 8.7?

@pebrc pebrc added >enhancement Enhancement of existing functionality v2.7.0 labels Mar 28, 2023
@botelastic botelastic bot removed the triage label Mar 28, 2023
@axw
Copy link
Member Author

axw commented Mar 28, 2023

will APM server 8.7 work without this i.e. is this a blocker for ECK + APM server 8.7?

It's not a blocker. There are two features that this affects. Those features previously required a kibanaRef, and they will continue to work with a kibanaRef. The reason behind this change is essentially to allow the features to work without a connection to Kibana.

@pebrc
Copy link
Collaborator

pebrc commented Mar 28, 2023

It's not a blocker.

That's good to hear. @thbkrkr I think we should still aim to include this given that we have a bit more time.

@thbkrkr
Copy link
Contributor

thbkrkr commented Mar 28, 2023

@thbkrkr I think we should still aim to include this given that we have a bit more time.

Yes, I think so too. I merge and take care of the backport.

@thbkrkr thbkrkr changed the title Add eck_apm_user_role_v87 role Add a new role for APM Server 8.7.0+ Mar 28, 2023
@thbkrkr thbkrkr merged commit 8aa4760 into elastic:main Mar 28, 2023
@axw axw deleted the apm-role-v87 branch March 28, 2023 13:17
thbkrkr pushed a commit to thbkrkr/cloud-on-k8s that referenced this pull request Mar 28, 2023
Add a new role for APM Server 8.7.0+, adding read privileges for the system indices
`.apm-agent-configuration` and `.apm-source-map`.
This change is essentially to allow features that previously required a kibanaRef to work
without a connection to Kibana.
thbkrkr added a commit that referenced this pull request Mar 28, 2023
Add a new role for APM Server 8.7.0+, adding read privileges for the system indices
`.apm-agent-configuration` and `.apm-source-map`.
This change is essentially to allow features that previously required a kibanaRef to work
without a connection to Kibana.

Co-authored-by: Andrew Wilkins <axwalk@gmail.com>
thbkrkr added a commit that referenced this pull request Apr 25, 2023
This adds a script that helps identifying changes that have been labeled as belonging to certain release version where the corresponding commits are not in the release branch. Its main purpose is to discover missing backports.

Testing:

- Success:

```sh
> hack/release-notes/check-release-notes.sh v2.7.0
Compare 'merged PR from 2.6.1 to v2.7.0' with 'release-notes/v2.7.0'
✅ LGTM
```

- Failure:

Simulate an invalid generated release note by deleting an entry:
```diff
diff --git a/docs/release-notes/2.7.0.asciidoc b/docs/release-notes/2.7.0.asciidoc
index 1518c41e6..ba3580da8 100644
--- a/docs/release-notes/2.7.0.asciidoc
+++ b/docs/release-notes/2.7.0.asciidoc
@@ -11,7 +11,6 @@
 [float]
 === Enhancements

-* Add a new role for APM Server 8.7.0+ {pull}6605[#6605]
+* Add a new role for APM Server 8.7.0+ {pull}16605[#6605]
```

```sh
> hack/release-notes/check-release-notes.sh v2.7.0
Compare 'merged PR from 2.6.1 to v2.7.0' with 'release-notes/v2.7.0'
❌ Error: no commit found for the following issues:
> #16605
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>enhancement Enhancement of existing functionality v2.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants