-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Fleet] Fix add fleet server flow from agent policy page #149507
Conversation
Pinging @elastic/fleet (Team:Fleet) |
@elasticmachine merge upstream |
@nchaulet I tested locally and I'm seeing some flickering in the flyout when clicking on "add agent": Flickering.movI remember having this issue when I was working on the flyout last year, it probably depends on the order which the flags are evaluated. |
…o fix-add-fleet-server
@criamico I just pushed a fix for the flickering case let me know if it's working for you |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚢
💚 Build Succeeded
Metrics [docs]Async chunks
History
To update your PR or re-run it, just comment with: cc @nchaulet |
) (cherry picked from commit da83d96)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…) (#149605) # Backport This will backport the following commits from `main` to `8.6`: - [[Fleet] Fix add fleet server flow from agent policy page (#149507)](#149507) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Nicolas Chaulet","email":"[email protected]"},"sourceCommit":{"committedDate":"2023-01-26T15:55:36Z","message":"[Fleet] Fix add fleet server flow from agent policy page (#149507)","sha":"da83d96ff65a3ad5f56d350576e6c4811e5f26af","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","v8.7.0","v8.6.1"],"number":149507,"url":"https://github.com/elastic/kibana/pull/149507","mergeCommit":{"message":"[Fleet] Fix add fleet server flow from agent policy page (#149507)","sha":"da83d96ff65a3ad5f56d350576e6c4811e5f26af"}},"sourceBranch":"main","suggestedTargetBranches":["8.6"],"targetPullRequestStates":[{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/149507","number":149507,"mergeCommit":{"message":"[Fleet] Fix add fleet server flow from agent policy page (#149507)","sha":"da83d96ff65a3ad5f56d350576e6c4811e5f26af"}},{"branch":"8.6","label":"v8.6.1","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}] BACKPORT--> Co-authored-by: Nicolas Chaulet <[email protected]>
Summary
Resolve #143519
Fix the add Fleet server flow from the agent policy flow
UI Changes
On click on
Add Fleet Server
Before
After