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.

Sign up for GitHub

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

Jump to bottom

[NET-7571] Ensure services function uses specified Consul namespace #1874

Merged
merged 6 commits into from
Feb 7, 2024
Merged

[NET-7571] Ensure services function uses specified Consul namespace #1874

merged 6 commits into from
Feb 7, 2024

Commits on Feb 6, 2024

  1. [NET-7571] 

    - bug fix for SeatGeek: Services function in CT (consul-template) does not update the namespace for each services API call
- opts should be defaulted to the opts at query creation.
- then they should be merged with the opts present in Fetch.
- currently the merge was the other way around.
- ENT test support is not there, so no test cases added as of now.
    @kkavish
    kkavish committed Feb 6, 2024
    Configuration menu
    Copy the full SHA
    031d0fb View commit details
    Browse the repository at this point in the history

  2. [NET-7571] 

    - bug fix for SeatGeek: Services function in CT (consul-template) does not update the namespace for each services API call
- opts should be defaulted to the opts at query creation.
- then they should be merged with the opts present in Fetch.
- currently the merge was the other way around.
- ENT test support is not there, so no test cases added as of now.
    @kkavish
    kkavish committed Feb 6, 2024
    Configuration menu
    Copy the full SHA
    9bd3d8e View commit details
    Browse the repository at this point in the history

  3. [NET-7571] 

    - bug fix for SeatGeek: Services function in CT (consul-template) does not update the namespace for each services API call
- opts should be defaulted to the opts at query creation.
- then they should be merged with the opts present in Fetch.
- currently the merge was the other way around.
- ENT test support is not there, so no test cases added as of now.
    @kkavish
    kkavish committed Feb 6, 2024
    Configuration menu
    Copy the full SHA
    b30687a View commit details
    Browse the repository at this point in the history

  4. [NET-7571] 

    - bug fix for SeatGeek: Services function in CT (consul-template) does not update the namespace for each services API call
- opts should be defaulted to the opts at query creation.
- then they should be merged with the opts present in Fetch.
- currently the merge was the other way around.
- ENT test support is not there, so no test cases added as of now.
    @kkavish
    kkavish committed Feb 6, 2024
    Configuration menu
    Copy the full SHA
    8855d4a View commit details
    Browse the repository at this point in the history

  5. [NET-7571] 

    - bug fix for SeatGeek: Services function in CT (consul-template) does not update the namespace for each services API call
- opts should be defaulted to the opts at query creation.
- then they should be merged with the opts present in Fetch.
- currently the merge was the other way around.
- ENT test support is not there, so no test cases added as of now.
    @kkavish
    kkavish committed Feb 6, 2024
    Configuration menu
    Copy the full SHA
    0ea26c5 View commit details
    Browse the repository at this point in the history

Commits on Feb 7, 2024

  1. [NET-7571] 

    - updating comments
    @kkavish
    kkavish committed Feb 7, 2024
    Configuration menu
    Copy the full SHA
    5b66da6 View commit details
    Browse the repository at this point in the history