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

[DOCS] Temporarily remove APM links #79411

Merged
merged 3 commits into from
Oct 19, 2021

Conversation

bmorelli25
Copy link
Member

Hey 👋

Big changes are coming to APM and the APM docs in 7.16. I'd like to temporarily remove these two links from the 7.16 and master branches of the Elasticsearch Reference so that I can get a docs PR to pass. These links (and the content) will be updated once I have new monitoring documentation in place.

For elastic/observability-docs#1073.
All broken links: elastic/docs#2239.

@bmorelli25 bmorelli25 added >docs General docs changes v7.16.0 labels Oct 19, 2021
@bmorelli25 bmorelli25 requested a review from a team October 19, 2021 01:31
@bmorelli25 bmorelli25 self-assigned this Oct 19, 2021
@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Oct 19, 2021
@elasticmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added v8.0.0 external-contributor Pull request authored by a developer outside the Elasticsearch team labels Oct 19, 2021
@jrodewig jrodewig changed the title docs: temporarily remove APM links [DOCS] Temporarily remove APM links Oct 19, 2021
@jrodewig jrodewig added the :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) label Oct 19, 2021
@elasticmachine elasticmachine added the Team:Security Meta label for security team label Oct 19, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

Copy link
Contributor

@jrodewig jrodewig left a comment

Choose a reason for hiding this comment

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

LGTM with one small change. @lockewritesdocs may want to take a look as this lands in the security area.

Copy link
Contributor

@lockewritesdocs lockewritesdocs left a comment

Choose a reason for hiding this comment

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

LGTM, with incorporating the change from @jrodewig. @bmorelli25, ping me on the PR that adds new links when you're ready for review.

@bmorelli25
Copy link
Member Author

@elasticmachine merge upstream

@bmorelli25 bmorelli25 merged commit d7fd4b5 into elastic:master Oct 19, 2021
@bmorelli25 bmorelli25 deleted the update-apm-links branch October 19, 2021 19:03
jrodewig added a commit that referenced this pull request Oct 19, 2021
Co-authored-by: James Rodewig <[email protected]>
Co-authored-by: Elastic Machine <[email protected]>
weizijun added a commit to weizijun/elasticsearch that referenced this pull request Oct 20, 2021
* upstream/master: (24 commits)
  Implement framework for migrating system indices (elastic#78951)
  Improve transient settings deprecation message (elastic#79504)
  Remove getValue and getValues from Field (elastic#79516)
  Store Template's mappings as bytes for disk serialization (elastic#78746)
  [ML] Add queue_capacity setting to start deployment API (elastic#79433)
  [ML] muting rest compat test issue elastic#79518 (elastic#79519)
  Avoid redundant available indices check (elastic#76540)
  Re-enable BWC tests
  TEST Ensure password 14 chars length on Kerberos FIPS tests (elastic#79496)
  [DOCS] Temporarily remove APM links (elastic#79411)
  Fix CCSDuelIT for skipped shards (elastic#79490)
  Add other time accounting in HotThreads (elastic#79392)
  Add deprecation info API entries for deprecated monitoring settings (elastic#78799)
  Add note in breaking changes for nameid_format (elastic#77785)
  Use 'migration' instead of 'upgrade' in GET system feature migration status responses (elastic#79302)
  Upgrade lucene version 8b68bf60c98 (elastic#79461)
  Use Strings#EMPTY_ARRAY (elastic#79452)
  Quicker shared cache file preallocation (elastic#79447)
  [ML] Removing some code that's obsolete for 8.0 (elastic#79444)
  Ensure indexing_data CCR requests are compressed (elastic#79413)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs General docs changes external-contributor Pull request authored by a developer outside the Elasticsearch team :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) Team:Docs Meta label for docs team Team:Security Meta label for security team v7.16.0 v8.0.0-beta1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants