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

Put environment back in uiFilters #92561

Merged
merged 1 commit into from
Feb 24, 2021
Merged

Put environment back in uiFilters #92561

merged 1 commit into from
Feb 24, 2021

Conversation

smith
Copy link
Contributor

@smith smith commented Feb 24, 2021

environment is still used in uiFilters in UX, but not anywhere else.

It was removed when removing environment from UI filters for the rest of APM and broke UX's environment switcher. Put it back.

`environment` is still used in `uiFilters` in UX, but not anywhere else.

It was removed when removing environment from UI filters for the rest of APM and broke UX's environment switcher. Put it back.
@smith smith added v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.12.0 auto-backport Deprecated - use backport:version if exact versions are needed labels Feb 24, 2021
@smith smith requested a review from a team as a code owner February 24, 2021 03:56
@botelastic botelastic bot added the Team:APM All issues that need APM UI Team support label Feb 24, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/apm-ui (Team:apm)

Copy link
Contributor

@ogupte ogupte left a comment

Choose a reason for hiding this comment

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

:shipit:

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 5.3MB 5.3MB +36.0B

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

@smith smith merged commit 06418ae into elastic:master Feb 24, 2021
@kibanamachine
Copy link
Contributor

💔 Backport failed

❌ 7.12: Commit could not be cherrypicked due to conflicts

To backport manually, check out the target branch and run:
node scripts/backport --pr 92561

smith added a commit to smith/kibana that referenced this pull request Feb 24, 2021
`environment` is still used in `uiFilters` in UX, but not anywhere else.

It was removed when removing environment from UI filters for the rest of APM and broke UX's environment switcher. Put it back.
# Conflicts:
#	x-pack/plugins/apm/public/context/url_params_context/url_params_context.tsx
smith added a commit to smith/kibana that referenced this pull request Feb 24, 2021
`environment` is still used in `uiFilters` in UX, but not anywhere else.

It was removed when removing environment from UI filters for the rest of APM and broke UX's environment switcher. Put it back.
# Conflicts:
#	x-pack/plugins/apm/public/context/url_params_context/url_params_context.tsx
smith added a commit that referenced this pull request Feb 24, 2021
`environment` is still used in `uiFilters` in UX, but not anywhere else.

It was removed when removing environment from UI filters for the rest of APM and broke UX's environment switcher. Put it back.
# Conflicts:
#	x-pack/plugins/apm/public/context/url_params_context/url_params_context.tsx
gmmorris added a commit to gmmorris/kibana that referenced this pull request Feb 25, 2021
…bana into task-manager/docs-monitoring

* 'task-manager/docs-monitoring' of github.com:gmmorris/kibana: (120 commits)
  Bump the CI Node.js version from 14.15.4 to 14.16.0 (elastic#92578)
  [Uptime] Search made easy (elastic#88581)
  [Lens] Stop requesting stats on Records field (elastic#92508)
  [APM] Correlations style polish in prep for release (elastic#92514)
  [Lens] Fix sorting undefined, null and NaN values (elastic#92575)
  Put environment back in uiFilters (elastic#92561)
  [Discover] Fix sorting by _score behavior (elastic#92132)
  Add metricbeat steps to jenkins_build_load_testing.sh (elastic#90290)
  [Usage Collection] Remove unused UI Metric APIs (elastic#91620)
  Fix timelion deprecation documentation links (elastic#92576)
  [TSVB] Enable `dual mode`, support index patterns and strings (elastic#92395)
  Fix text align in Safari (elastic#92374)
  [Actions][Doc] Clean up Actions README (elastic#91789)
  chore(NA): add missing files to exclude when building bazel prod packages (elastic#92506)
  [Security Solution][Endpoint][Admin] Match Policy Details in Security Solution with Fleet (elastic#92047)
  Revert "[build] Add task skips intended for partial builds (elastic#92076)"
  [Enterprise Search] Change icon for error in EuiCallouts (elastic#92527)
  [Alerts][Docs] Alert types doc update. Added refs to applications specific alerts groups. (elastic#91787)
  [Security Solution][Exceptions] - Update exceptions modal to use existing lists plugin useApi hook (elastic#92348)
  [DOCS] Updates refresh text in index pattern doc (elastic#92469)
  ...
@kibanamachine
Copy link
Contributor

Looks like this PR has backport PRs but they still haven't been merged. Please merge them ASAP to keep the branches relatively in sync.

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Feb 26, 2021
smith added a commit that referenced this pull request Feb 27, 2021
`environment` is still used in `uiFilters` in UX, but not anywhere else.

It was removed when removing environment from UI filters for the rest of APM and broke UX's environment switcher. Put it back.
# Conflicts:
#	x-pack/plugins/apm/public/context/url_params_context/url_params_context.tsx
@kibanamachine kibanamachine removed the backport missing Added to PRs automatically when the are determined to be missing a backport. label Feb 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed release_note:skip Skip the PR/issue when compiling release notes Team:APM All issues that need APM UI Team support v7.12.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants