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

[Request] Remove Functionbeat from the kbn/doc-links package #193030

Closed
kilfoyle opened this issue Sep 16, 2024 · 4 comments · Fixed by #198902
Closed

[Request] Remove Functionbeat from the kbn/doc-links package #193030

kilfoyle opened this issue Sep 16, 2024 · 4 comments · Fixed by #198902
Assignees

Comments

@kilfoyle
Copy link
Contributor

I'm trying to freeze the Functionbeat documentation as the product is fully deprecated.

In the elastic/docs repo, my pull request to freeze those docs fails our CI link checker, with an error complaining about an entry in packages/kbn-doc-links/src/get_doc_links.ts:

2024-09-10 10:28:42 EDT | INFO:build_docs:Bad cross-document links:
-- | --
  | 2024-09-10 10:28:42 EDT | INFO:build_docs:  Kibana [master]: packages/kbn-doc-links/src/get_doc_links.ts contains broken links to:
  | 2024-09-10 10:28:42 EDT | INFO:build_docs:   - en/beats/functionbeat/master
  | 2024-09-10 10:29:31 EDT | 🚨 Error: The command exited with status 255

I opened this kibana PR to remove the relevant lines from the packages/kbn-doc-links/src/get_doc_links.ts file, but (as shown in that PR) the change failed the Kibana CI checks. I have no familiarity with that get_doc_links.ts file so I'd really appreciate if anyone can help remove the Functionbeat references, so that I can in turn can Freeze the Functionbeat docs at the current release (8.15 as of today).

@botelastic botelastic bot added the needs-team Issues missing a team label label Sep 16, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-docs (Team:Docs)

@botelastic botelastic bot removed the needs-team Issues missing a team label label Sep 18, 2024
@lcawl lcawl self-assigned this Sep 18, 2024
@lcawl
Copy link
Contributor

lcawl commented Sep 18, 2024

I think the only thing missing doc link service wise from #192491 was to also remove that functionbeat declaration from the types.ts file. For example here:

readonly functionbeat: {

Before it can be removed, however, per DeDe's comments in elastic/docs#2895, the places in the UI that use that variable will need to be updated. From my searches, it's used here: https://github.com/elastic/kibana/blob/main/src/plugins/home/public/application/components/tutorial/replace_template_strings.js

... that config.docs.beats.functionbeat definition in turn is used in https://github.com/elastic/kibana/blob/main/src/plugins/home/server/tutorials/cloudwatch_logs/index.ts and https://github.com/elastic/kibana/blob/main/src/plugins/home/server/tutorials/instructions/functionbeat_instructions.ts

It looks like someone was trying to clean those out recently per #172909 so they might be able to help with the removal.

@lcawl
Copy link
Contributor

lcawl commented Sep 18, 2024

If those Uis need to still point to Functionbeat docs, one solution might be to simply update the doc link service (in all branches after 8.15) to simply point to the last version of those docs. For example:

 functionbeat: {
      base: `${ELASTIC_WEBSITE_URL}guide/en/beats/functionbeat/8.15}`,
    },

@kilfoyle
Copy link
Contributor Author

kilfoyle commented Sep 25, 2024

Thanks for the detective work @lcawl! This helps a lot.

I've opened a new issue specifically for the Integrations Team, since one remaining link appears to come from a Cloud Watch tutorial that uses Functionbeat.

kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Nov 6, 2024
## Summary

Remove docs link to functionbeat as it has been deprecated and will be
removed in 9.0

### Checklist

Delete any items that are not applicable to this PR.

### Risk Matrix

Delete this section if it is not applicable to this PR.

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
- [ ] This will appear in the **Release Notes** and follow the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

### Related Issues:
 - Closes elastic#193030
 - Relates elastic/beats#40745

(cherry picked from commit 4e753fa)
kilfoyle pushed a commit to kilfoyle/kibana that referenced this issue Nov 6, 2024
## Summary

Remove docs link to functionbeat as it has been deprecated and will be
removed in 9.0

### Checklist

Delete any items that are not applicable to this PR.


### Risk Matrix

Delete this section if it is not applicable to this PR.

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
- [ ] This will appear in the **Release Notes** and follow the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

### Related Issues:
 - Closes elastic#193030
 - Relates elastic/beats#40745
mgadewoll pushed a commit to mgadewoll/kibana that referenced this issue Nov 7, 2024
## Summary

Remove docs link to functionbeat as it has been deprecated and will be
removed in 9.0

### Checklist

Delete any items that are not applicable to this PR.


### Risk Matrix

Delete this section if it is not applicable to this PR.

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
- [ ] This will appear in the **Release Notes** and follow the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

### Related Issues:
 - Closes elastic#193030
 - Relates elastic/beats#40745
AndersonQ added a commit that referenced this issue Nov 13, 2024
## Summary

Remove functionbeat tutorial and translations. It's been deprecated and won't be
shipped on 9.0

### Checklist

Delete any items that are not applicable to this PR.

### Risk Matrix

I'm not sure, its just deleting code/documentation

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
- [ ] This will appear in the **Release Notes** and follow the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

## Related issues

- Relates elastic/beats#40745
- #193030
AndersonQ added a commit to AndersonQ/kibana that referenced this issue Nov 13, 2024
## Summary

Remove functionbeat tutorial and translations. It's been deprecated and won't be
shipped on 9.0

### Checklist

Delete any items that are not applicable to this PR.

### Risk Matrix

I'm not sure, its just deleting code/documentation

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
- [ ] This will appear in the **Release Notes** and follow the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

## Related issues

- Relates elastic/beats#40745
- elastic#193030

(cherry picked from commit 710c4cc)

# Conflicts:
#	api_docs/telemetry.devdocs.json
#	x-pack/plugins/translations/translations/zh-CN.json
AndersonQ added a commit to AndersonQ/kibana that referenced this issue Nov 13, 2024
## Summary

Remove functionbeat tutorial and translations. It's been deprecated and won't be
shipped on 9.0

### Checklist

Delete any items that are not applicable to this PR.

### Risk Matrix

I'm not sure, its just deleting code/documentation

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
- [ ] This will appear in the **Release Notes** and follow the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

## Related issues

- Relates elastic/beats#40745
- elastic#193030

(cherry picked from commit 710c4cc)

# Conflicts:
#	api_docs/telemetry.devdocs.json
#	x-pack/plugins/translations/translations/zh-CN.json
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Nov 18, 2024
## Summary

Remove functionbeat tutorial and translations. It's been deprecated and won't be
shipped on 9.0

### Checklist

Delete any items that are not applicable to this PR.

### Risk Matrix

I'm not sure, its just deleting code/documentation

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
- [ ] This will appear in the **Release Notes** and follow the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

## Related issues

- Relates elastic/beats#40745
- elastic#193030
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this issue Nov 18, 2024
## Summary

Remove functionbeat tutorial and translations. It's been deprecated and won't be
shipped on 9.0

### Checklist

Delete any items that are not applicable to this PR.

### Risk Matrix

I'm not sure, its just deleting code/documentation

### For maintainers

- [ ] This was checked for breaking API changes and was [labeled
appropriately](https://www.elastic.co/guide/en/kibana/master/contributing.html#_add_your_labels)
- [ ] This will appear in the **Release Notes** and follow the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

## Related issues

- Relates elastic/beats#40745
- elastic#193030
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants