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

[Fleet][Cloud] Enrollment token table may show an empty last page #167663 #188049

Merged
merged 3 commits into from
Jul 11, 2024
Merged

[Fleet][Cloud] Enrollment token table may show an empty last page #167663 #188049

merged 3 commits into from
Jul 11, 2024

Conversation

pra2107tham
Copy link
Contributor

@pra2107tham pra2107tham commented Jul 11, 2024

Fixes #167663

Summary

Fixes the creation of extra empty page at pagination of 5 rows

Video

Recording.2024-07-11.120546.mp4

@pra2107tham pra2107tham requested a review from a team as a code owner July 11, 2024 06:47
@botelastic botelastic bot added the Team:Fleet Team label for Observability Data Collection Fleet team label Jul 11, 2024
Copy link

cla-checker-service bot commented Jul 11, 2024

💚 CLA has been signed

@elasticmachine
Copy link
Contributor

Pinging @elastic/fleet (Team:Fleet)

@jlind23
Copy link
Contributor

jlind23 commented Jul 11, 2024

@pra2107tham could you please sgn the contributor agreement as stated in this comment?

@pra2107tham
Copy link
Contributor Author

@pra2107tham could you please sgn the contributor agreement as stated in this comment?

I signed the contributer agreement, whats next?

@jlind23
Copy link
Contributor

jlind23 commented Jul 11, 2024

@pra2107tham next step would be to have someone reviewing your work.

Copy link
Contributor

@juliaElastic juliaElastic left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for contributing!

@juliaElastic
Copy link
Contributor

/ci

@juliaElastic
Copy link
Contributor

@elasticmachine run elasticsearch-ci/docs

@pra2107tham
Copy link
Contributor Author

@elasticmachine run elasticsearch-ci/docs

How?

@juliaElastic
Copy link
Contributor

buildkite test this

@pra2107tham
Copy link
Contributor Author

buildkite test this

do I have to do this?

@juliaElastic
Copy link
Contributor

@elasticmachine run buildkite/docs-build-pr

@juliaElastic
Copy link
Contributor

juliaElastic commented Jul 11, 2024

I'm trying to trigger the docs build step which is not started yet. Something changed since the last time I've used these commands.
It looks like the last one worked.

@pra2107tham
Copy link
Contributor Author

I'm trying to trigger the docs build step which is not started yet. Something changed since the last time I've used these commands. It looks like the last one worked.

Ohh okay, so I am done with my task ig

@pra2107tham
Copy link
Contributor Author

@elasticmachine run kibana-ci

@pra2107tham
Copy link
Contributor Author

@juliaElastic can you complete all the remaining checks?

@juliaElastic
Copy link
Contributor

The kibana-ci is still running, when completed, we will be ready to merge the pr.

@juliaElastic juliaElastic enabled auto-merge (squash) July 11, 2024 09:01
@juliaElastic juliaElastic merged commit 1f7fdfd into elastic:main Jul 11, 2024
21 checks passed
@kibanamachine kibanamachine added v8.16.0 backport:skip This commit does not require backporting labels Jul 11, 2024
@elasticmachine
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
fleet 1.8MB 1.8MB +11.0B

@jen-huang jen-huang removed the backport:skip This commit does not require backporting label Jul 11, 2024
@jen-huang jen-huang added v8.15.0 backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Jul 11, 2024
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Jul 11, 2024
…stic#167663 (elastic#188049)

Fixes elastic#167663

## Summary

Fixes the creation of extra empty page at pagination of 5 rows

### Video

https://github.com/elastic/kibana/assets/140709379/cdac6a22-f239-4b70-b40c-93fc727418a6

Co-authored-by: Julien Lind <[email protected]>
(cherry picked from commit 1f7fdfd)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.15

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jul 11, 2024
…age #167663 (#188049) (#188127)

# Backport

This will backport the following commits from `main` to `8.15`:
- [[Fleet][Cloud] Enrollment token table may show an empty last page
#167663 (#188049)](#188049)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Pratham
Shirbhate","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-07-11T09:15:54Z","message":"[Fleet][Cloud]
Enrollment token table may show an empty last page #167663
(#188049)\n\nFixes #167663 \r\n\r\n## Summary\r\n\r\nFixes the creation
of extra empty page at pagination of 5 rows\r\n\r\n\r\n### Video
\r\n\r\n\r\nhttps://github.com/elastic/kibana/assets/140709379/cdac6a22-f239-4b70-b40c-93fc727418a6\r\n\r\nCo-authored-by:
Julien Lind
<[email protected]>","sha":"1f7fdfdfe428525b08bf14321463d091bdad4c79","branchLabelMapping":{"^v8.16.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:fix","💝community","Team:Fleet","backport:prev-minor","v8.15.0","v8.16.0"],"title":"[Fleet][Cloud]
Enrollment token table may show an empty last page
#167663","number":188049,"url":"https://github.com/elastic/kibana/pull/188049","mergeCommit":{"message":"[Fleet][Cloud]
Enrollment token table may show an empty last page #167663
(#188049)\n\nFixes #167663 \r\n\r\n## Summary\r\n\r\nFixes the creation
of extra empty page at pagination of 5 rows\r\n\r\n\r\n### Video
\r\n\r\n\r\nhttps://github.com/elastic/kibana/assets/140709379/cdac6a22-f239-4b70-b40c-93fc727418a6\r\n\r\nCo-authored-by:
Julien Lind
<[email protected]>","sha":"1f7fdfdfe428525b08bf14321463d091bdad4c79"}},"sourceBranch":"main","suggestedTargetBranches":["8.15"],"targetPullRequestStates":[{"branch":"8.15","label":"v8.15.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.16.0","branchLabelMappingKey":"^v8.16.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/188049","number":188049,"mergeCommit":{"message":"[Fleet][Cloud]
Enrollment token table may show an empty last page #167663
(#188049)\n\nFixes #167663 \r\n\r\n## Summary\r\n\r\nFixes the creation
of extra empty page at pagination of 5 rows\r\n\r\n\r\n### Video
\r\n\r\n\r\nhttps://github.com/elastic/kibana/assets/140709379/cdac6a22-f239-4b70-b40c-93fc727418a6\r\n\r\nCo-authored-by:
Julien Lind
<[email protected]>","sha":"1f7fdfdfe428525b08bf14321463d091bdad4c79"}}]}]
BACKPORT-->

Co-authored-by: Pratham Shirbhate <[email protected]>
jen-huang added a commit to jen-huang/kibana that referenced this pull request Sep 27, 2024
jen-huang added a commit that referenced this pull request Sep 30, 2024
…page #167663 (#188049)" (#194343)

## Summary

Resolves #194339.

This PR reverts #188049 as it inadvertently broke pagination of the
enrollment tokens page. This means that #167663 will need to be
re-opened and fixed in a different way.
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 30, 2024
…page elastic#167663 (elastic#188049)" (elastic#194343)

## Summary

Resolves elastic#194339.

This PR reverts elastic#188049 as it inadvertently broke pagination of the
enrollment tokens page. This means that elastic#167663 will need to be
re-opened and fixed in a different way.

(cherry picked from commit 283f6e6)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 30, 2024
…page elastic#167663 (elastic#188049)" (elastic#194343)

## Summary

Resolves elastic#194339.

This PR reverts elastic#188049 as it inadvertently broke pagination of the
enrollment tokens page. This means that elastic#167663 will need to be
re-opened and fixed in a different way.

(cherry picked from commit 283f6e6)
kibanamachine added a commit that referenced this pull request Sep 30, 2024
…empty last page #167663 (#188049)&quot; (#194343) (#194480)

# Backport

This will backport the following commits from `main` to `8.x`:
- [Revert &quot;[Fleet][Cloud] Enrollment token table may show an empty
last page #167663 (#188049)&quot;
(#194343)](#194343)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Jen
Huang","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-30T19:35:13Z","message":"Revert
\"[Fleet][Cloud] Enrollment token table may show an empty last page
#167663 (#188049)\" (#194343)\n\n## Summary\r\n\r\nResolves
#194339.\r\n\r\nThis PR reverts #188049 as it inadvertently broke
pagination of the\r\nenrollment tokens page. This means that #167663
will need to be\r\nre-opened and fixed in a different
way.","sha":"283f6e64c9656c97fa37be34f3e6fab2e8ad459f","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","v9.0.0","backport:prev-major"],"title":"Revert
\"[Fleet][Cloud] Enrollment token table may show an empty last page
#167663
(#188049)\"","number":194343,"url":"https://github.com/elastic/kibana/pull/194343","mergeCommit":{"message":"Revert
\"[Fleet][Cloud] Enrollment token table may show an empty last page
#167663 (#188049)\" (#194343)\n\n## Summary\r\n\r\nResolves
#194339.\r\n\r\nThis PR reverts #188049 as it inadvertently broke
pagination of the\r\nenrollment tokens page. This means that #167663
will need to be\r\nre-opened and fixed in a different
way.","sha":"283f6e64c9656c97fa37be34f3e6fab2e8ad459f"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194343","number":194343,"mergeCommit":{"message":"Revert
\"[Fleet][Cloud] Enrollment token table may show an empty last page
#167663 (#188049)\" (#194343)\n\n## Summary\r\n\r\nResolves
#194339.\r\n\r\nThis PR reverts #188049 as it inadvertently broke
pagination of the\r\nenrollment tokens page. This means that #167663
will need to be\r\nre-opened and fixed in a different
way.","sha":"283f6e64c9656c97fa37be34f3e6fab2e8ad459f"}}]}] BACKPORT-->

Co-authored-by: Jen Huang <[email protected]>
kibanamachine added a commit that referenced this pull request Sep 30, 2024
… empty last page #167663 (#188049)&quot; (#194343) (#194479)

# Backport

This will backport the following commits from `main` to `8.15`:
- [Revert &quot;[Fleet][Cloud] Enrollment token table may show an empty
last page #167663 (#188049)&quot;
(#194343)](#194343)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Jen
Huang","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-09-30T19:35:13Z","message":"Revert
\"[Fleet][Cloud] Enrollment token table may show an empty last page
#167663 (#188049)\" (#194343)\n\n## Summary\r\n\r\nResolves
#194339.\r\n\r\nThis PR reverts #188049 as it inadvertently broke
pagination of the\r\nenrollment tokens page. This means that #167663
will need to be\r\nre-opened and fixed in a different
way.","sha":"283f6e64c9656c97fa37be34f3e6fab2e8ad459f","branchLabelMapping":{"^v9.0.0$":"main","^v8.16.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Team:Fleet","v9.0.0","backport:prev-major"],"title":"Revert
\"[Fleet][Cloud] Enrollment token table may show an empty last page
#167663
(#188049)\"","number":194343,"url":"https://github.com/elastic/kibana/pull/194343","mergeCommit":{"message":"Revert
\"[Fleet][Cloud] Enrollment token table may show an empty last page
#167663 (#188049)\" (#194343)\n\n## Summary\r\n\r\nResolves
#194339.\r\n\r\nThis PR reverts #188049 as it inadvertently broke
pagination of the\r\nenrollment tokens page. This means that #167663
will need to be\r\nre-opened and fixed in a different
way.","sha":"283f6e64c9656c97fa37be34f3e6fab2e8ad459f"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/194343","number":194343,"mergeCommit":{"message":"Revert
\"[Fleet][Cloud] Enrollment token table may show an empty last page
#167663 (#188049)\" (#194343)\n\n## Summary\r\n\r\nResolves
#194339.\r\n\r\nThis PR reverts #188049 as it inadvertently broke
pagination of the\r\nenrollment tokens page. This means that #167663
will need to be\r\nre-opened and fixed in a different
way.","sha":"283f6e64c9656c97fa37be34f3e6fab2e8ad459f"}}]}] BACKPORT-->

Co-authored-by: Jen Huang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) 💝community release_note:fix Team:Fleet Team label for Observability Data Collection Fleet team v8.15.0 v8.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fleet][Cloud] Enrollment token table may show an empty last page
6 participants