Skip to content

Feature implementation from commits 19ca140..f12a82a#2

Open
yashuatla wants to merge 15 commits intofeature-base-2from
feature-head-2
Open

Feature implementation from commits 19ca140..f12a82a#2
yashuatla wants to merge 15 commits intofeature-base-2from
feature-head-2

Conversation

@yashuatla
Copy link
Owner

This PR contains changes from a range of commits from the original repository.

Commit Range: 19ca140..f12a82a
Files Changed: 74 (31 programming files)
Programming Ratio: 41.9%

Commits included:

Chris S. Kim and others added 15 commits March 14, 2024 18:45
* disable scan for "GHSA-jw44-4f3j-q396" until patch fix in helm v3

* addressed comments
…ion to synced consul service (hashicorp#3693)

* Use EndpointSlice and propagate zone metadata to consul service

* Fix tests

* Add test for zone metadata

* Cleanup and changelog entry

* Fix clusterrole permissions and type on Informer

* Include region info for NodePort services

* Include topology region for all service types

* Update release note

* Fix tests

* fix sync-catalog-clusterrole and tests

* fix stash conflict

* adding endpoints permission back to sync catalog since it still uses it.

* Fix endpointslice map

* Fix topology region

* Remove region lookups, remove endpoints permissions, use pointers for endpointslice map

* Drop region test

---------

Co-authored-by: John Murret <john.murret@hashicorp.com>
…ashicorp#3718)

* Removes Legacy API Gateway Stanza that was deprecated in Consul 1.16
…icorp#3794)

In hashicorp#1527, we added support for OpenShift and Multus, which meant that the
`consul-cni` plugin was no longer necessarily the final CNI plugin run. While
working on a patch to allow compatibility with Nomad transparent proxy, I
discovered we'd never removed a now-failing unit test of the plugin for the
validation step. It looks like the remaining unit tests still cover the
remaining validation, so we can safely remove this test.

Ref: hashicorp#1527
Ref: hashicorp/nomad#10628
* Reorder gateway policy and role creation to avoid error messages in consul when policy/role already exists

* refactor for readability

* fix spacing

* Added changelog
* improve reliability of acceptance tests

* remove update to timeout

* add output to error
* First metrics pass

* Fix up build

* move to non-deprecated chart options

* Fix up charts and defaults

* Add changelog

* Fix bad merge

* Fix test

* fix linter error

* Fix extra yaml block from bad merge

* Switch == true check to use ParseBool
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants