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

Backport of Cc 5545: Upgrade HDS packages and modifiers into release/1.17.x #19245

Conversation

hc-github-team-consul-core
Copy link
Collaborator

Backport

This PR is auto-generated from #19226 to be assessed for backporting due to the inclusion of the label backport/1.17.

The below text is copied from the body of the original PR.


#19210

Trying to redo this PR but without all of the whitespace changes


Description from other PR

Description

Upgrades HDS packages to latest releases ahead of work for adding the HDS Side Nav to the UI.

Upgrading @hashicorp/design-system-components caused ember-modifier to resolve to a 4.x version which was not compatible with a bunch of our class modifiers as well as one our packages that was outdated (ember-in-viewport). Due to this I explicitly updated consul-ui to use the latest ember-modifier and ember-in-viewport packages.

Because of the above I had to go through and update all the class based modifiers to use the new modifier structure.

Upgraded ember-on-resize-modifier to fix issue where routing edge rects weren't getting set.

A nice video where I find out routing of all things has an issue at the end:

(this ends with me finding an issue with routing edges which I fixed after, see next screenshot)

video1069237399.mp4
Screenshot 2023-10-16 at 10 15 46 AM Screenshot 2023-10-13 at 4 08 05 PM Screenshot 2023-10-13 at 4 06 01 PM Screenshot 2023-10-13 at 4 05 32 PM Screenshot 2023-10-13 at 4 05 03 PM Screenshot 2023-10-13 at 3 42 20 PM Screenshot 2023-10-13 at 3 40 38 PM

Testing & Reproduction steps

Links

CC-5545

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

Overview of commits

rboyer and others added 30 commits October 13, 2023 15:32
Update version file to 1.18-dev, and replace 1.13 nightly test with
1.17.
This change adds ACL hooks to the remaining catalog and mesh resources, excluding any computed ones. Those will for now continue using the default operator:x permissions.

It refactors a lot of the common testing functions so that they can be re-used between resources.

There are also some types that we don't yet support (e.g. virtual IPs) that this change adds ACL hooks to for future-proofing.
…19187)

* NET-5073 - ProxyConfiguration: implement various connection options

* PR feedback - LocalConnection and InboundConnection do not affect exposed routes. configure L7 route destinations. fix connection proto sequence numbers.

* add timeout to L7 Route Destinations
Co-authored-by: Ronald Ekambi <ronekambi@gmail.com>
Enhance the xRoute ACL hook tests to cover tenanted situations.
These tests will only execute in enterprise.
Enhance the DestinationPolicy ACL hook tests to cover tenanted situations.
These tests will only execute in enterprise.
@hc-github-team-consul-core hc-github-team-consul-core requested a review from a team as a code owner October 17, 2023 13:28
@hc-github-team-consul-core hc-github-team-consul-core force-pushed the backport/CC-5545/no-big-changes-for-tylers-pr/violently-suitable-caiman branch from f19daa7 to 7be8f75 Compare October 17, 2023 13:28
@hc-github-team-consul-core hc-github-team-consul-core requested a review from a team October 17, 2023 13:28
@hc-github-team-consul-core hc-github-team-consul-core removed the request for review from a team October 17, 2023 13:28
@hc-github-team-consul-core hc-github-team-consul-core force-pushed the backport/CC-5545/no-big-changes-for-tylers-pr/violently-suitable-caiman branch from 3ae1ad9 to 9d9714e Compare October 17, 2023 13:28
Copy link
Collaborator

Choose a reason for hiding this comment

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

Auto approved Consul Bot automated PR

@github-actions github-actions bot added type/docs Documentation needs to be created/updated/clarified theme/api Relating to the HTTP API interface theme/cli Flags and documentation for the CLI interface theme/ui Anything related to the UI type/ci Relating to continuous integration (CI) tooling for testing or releases pr/dependencies PR specifically updates dependencies of project theme/contributing Additions and enhancements to community contributing materials labels Oct 17, 2023
@hc-github-team-consul-core hc-github-team-consul-core merged commit bf42dd6 into release/1.17.x Oct 17, 2023
94 checks passed
@hc-github-team-consul-core hc-github-team-consul-core deleted the backport/CC-5545/no-big-changes-for-tylers-pr/violently-suitable-caiman branch October 17, 2023 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/dependencies PR specifically updates dependencies of project theme/api Relating to the HTTP API interface theme/cli Flags and documentation for the CLI interface theme/contributing Additions and enhancements to community contributing materials theme/ui Anything related to the UI type/ci Relating to continuous integration (CI) tooling for testing or releases type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants