-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
UI: Replication page navigation fix #26325
Conversation
…cationAttrs which wasn't triggering component updates
Build Results: |
CI Results: |
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.
looks great to me!
@@ -38,7 +38,9 @@ export default Component.extend(ReplicationActions, DEFAULTS, { | |||
initialReplicationMode: null, | |||
cluster: null, | |||
|
|||
replicationAttrs: alias('cluster.replicationAttrs'), |
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.
wow, i totally forgot about alias
ever being a thing. it's like time traveling! 😛
* Add replication mirage handler * Add test with skipped failed assertion * Use component-calculated attrsForCurrentMode instead of cluster.replicationAttrs which wasn't triggering component updates * assert previously-skipped assertion * Add changelog
* Add replication mirage handler * Add test with skipped failed assertion * Use component-calculated attrsForCurrentMode instead of cluster.replicationAttrs which wasn't triggering component updates * assert previously-skipped assertion * Add changelog
* Add replication mirage handler * Add test with skipped failed assertion * Use component-calculated attrsForCurrentMode instead of cluster.replicationAttrs which wasn't triggering component updates * assert previously-skipped assertion * Add changelog
* Add replication mirage handler * Add test with skipped failed assertion * Use component-calculated attrsForCurrentMode instead of cluster.replicationAttrs which wasn't triggering component updates * assert previously-skipped assertion * Add changelog
* Add replication mirage handler * Add test with skipped failed assertion * Use component-calculated attrsForCurrentMode instead of cluster.replicationAttrs which wasn't triggering component updates * assert previously-skipped assertion * Add changelog
….15.x (#26143) * Change minimum retention window CE changes (#26118) * Retention window oss changes * latest oss changes * remove operator_diagnose change * backport of commit da21b85 (#25666) Co-authored-by: Scott Miller <[email protected]> * headers only modified if we have a header formatter and headers (#26140) * backport of commit 1885f16 (#26153) Co-authored-by: miagilepner <[email protected]> * Known issues: Vault Enterprise - Performance Standby nodes audit log all request headers (#26158) (#26159) * Add known issue docs for Ent Perf Standby audit header logging issue * attempt to improve description Co-authored-by: Peter Wilson <[email protected]> * Correct version for next 1.15 release (#26212) * Update CHANGELOG.md (#26215) To follow new processes for creating release notes on GitHub, I need to update the changelog on the release branch. I've opted to copy the entirety of the 1.15 changelog content to the release branch, adding the notes for 1.15.7. Next I'll create the tag per these instructions https://github.com/hashicorp/engineering-docs/blob/main/consul/releases/release-process.md#manually-create-github-releases-for-ent-only-patch-releases, and update the release notes for that tag & release. * backport of commit 92c5847 (#26234) Co-authored-by: Theron Voran <[email protected]> * backport of commit f1922d2 (#26272) Co-authored-by: Socheat Sok <[email protected]> * UI: Don't show Resultant-ACL banner when wildcard policy present (#26233) (#26271) * Add wildcard calc helpers to permissions service with tests * Check for wildcard access when calculating permissionsBanner * Move resultant-acl banner within TokenExpireWarning so it's mutually exclusive with token expired banner * fix permissions banner if statement * Add margin to resultant-acl * cleanup comments * backport of commit d1fda88 (#26302) Co-authored-by: James Bayer <[email protected]> * backport of commit 02312cb (#26305) Co-authored-by: Victor Rodriguez <[email protected]> * backport of commit c9dafc1 (#26187) Co-authored-by: NikolaiMagicnet <[email protected]> * UI: Replication page navigation fix (#26325) (#26339) * Add replication mirage handler * Add test with skipped failed assertion * Use component-calculated attrsForCurrentMode instead of cluster.replicationAttrs which wasn't triggering component updates * assert previously-skipped assertion * Add changelog * UI: fix replication nav 1.15.x (#26349) * Update test selectors specific to 1.15.x * calculate attrs based on replication-mode service instead of cluster model getter * backport of commit 71758f4 (#26358) Co-authored-by: Ryan Cragun <[email protected]> * UI: Dependency bumps 1.15.x (#26371) * reform yarn.lock without minimatch or qs in resolutions * pin async and nth-check * fix TS errors after bump * bump ember-template-lint and disable broken rules * pin ansi-html * add extra lint rule to skip * remove ember-d3 in favor of specific d3 libraries we import except d3-selection which was failing in compareAttributes * add changelog from PR to main --------- Co-authored-by: akshya96 <[email protected]> Co-authored-by: Scott Miller <[email protected]> Co-authored-by: Peter Wilson <[email protected]> Co-authored-by: miagilepner <[email protected]> Co-authored-by: Meggie <[email protected]> Co-authored-by: Theron Voran <[email protected]> Co-authored-by: Socheat Sok <[email protected]> Co-authored-by: Chelsea Shaw <[email protected]> Co-authored-by: James Bayer <[email protected]> Co-authored-by: Victor Rodriguez <[email protected]> Co-authored-by: NikolaiMagicnet <[email protected]> Co-authored-by: Ryan Cragun <[email protected]>
When a cluster is the replication primary for both
DR
andPerformance
types, navigating between the pages would not update the page data without a full page refresh. This PR fixes that behavior and adds a test.Before
![Before - replication pages](https://private-user-images.githubusercontent.com/82459713/320995926-3a986237-6fdd-47a6-a47e-786b8ce99abf.gif?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjU2Nzg1MzcsIm5iZiI6MTcyNTY3ODIzNywicGF0aCI6Ii84MjQ1OTcxMy8zMjA5OTU5MjYtM2E5ODYyMzctNmZkZC00N2E2LWE0N2UtNzg2YjhjZTk5YWJmLmdpZj9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MDclMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTA3VDAzMDM1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTI0ZWQ5NGMyNjhhYjg1ZWYyNWQyZWFhYjUwYjJiNDU4NDE5NmIzZWUyMmMzNThlMWI0ODAyNTFiMDJjMzRhOWEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.9GDsphv7P-rnr4ldZckwRYpDvO0U1YeUIfxjpf9yVrA)
After
![After - replication pages](https://private-user-images.githubusercontent.com/82459713/320997492-7459408e-e5e9-4aab-8c70-c673427bfda0.gif?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjU2Nzg1MzcsIm5iZiI6MTcyNTY3ODIzNywicGF0aCI6Ii84MjQ1OTcxMy8zMjA5OTc0OTItNzQ1OTQwOGUtZTVlOS00YWFiLThjNzAtYzY3MzQyN2JmZGEwLmdpZj9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA5MDclMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwOTA3VDAzMDM1N1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTYxMWU5YmY1YzEwYTM2ZDc3ZTQ4YzY1MDhjNmU4ZmIxNWI2NWQ5NTI0YTk3ZWU0ZTczYWIxYjk3MzEwNWY3N2ImWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.W5xHDTYBCkJMjSFqwFvoBhUINFWm0KClXhA2lFmlF0o)
We were relying on a calculated value from the cluster model to update the components, but it wasn't triggering properly. By moving the data to the parent component which uses it, the DOM changes are triggered.