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

tests: egressgw: minor improvements and clean ups #2063

Merged
merged 5 commits into from
Jan 5, 2024
Merged

tests: egressgw: minor improvements and clean ups #2063

merged 5 commits into from
Jan 5, 2024

Commits on Jan 5, 2024

  1. check: add ExcludedCIDRsKind enum 

    to better document how the ExcludedCIDRs param of
CiliumEgressGatewayPolicyParams works

Signed-off-by: Gilberto Bertin <[email protected]>
    @jibi
    jibi committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    68fb2dc View commit details
    Browse the repository at this point in the history

  2. check: allow to configure pod selector in WithCiliumEgressPolicy() 

    add a new PodSelectorKind parameter to CiliumEgressGatewayPolicyParams,
which allows to select the pod matching the "kind" label

Signed-off-by: Gilberto Bertin <[email protected]>
    @jibi
    jibi committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    47772c4 View commit details
    Browse the repository at this point in the history

  3. check: don't pass policy YAML to WithCiliumEgressPolicy() 

    As CiliumEgressGatewayPolicyParams allows now to use a single policy
template across all test scenarios, stop passing the same policy
template to all invocations of WithCiliumEgressPolicy()

Signed-off-by: Gilberto Bertin <[email protected]>
    @jibi
    jibi committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    c58c85f View commit details
    Browse the repository at this point in the history

  4. tests: egressgw: export WatiForEgressGatewayBpfPolicyEntries 

    and make it available to hooks

Signed-off-by: Gilberto Bertin <[email protected]>
    @jibi
    jibi committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    162da0a View commit details
    Browse the repository at this point in the history

  5. check: rename ExcludedCIDRs to ExcludedCIDRsConf 

    rename the ExcludedCIDRs CiliumEgressGatewayPolicyParams field to
ExcludedCIDRsConf to make it more explicit its use case

Signed-off-by: Gilberto Bertin <[email protected]>
    @jibi
    jibi committed Jan 5, 2024
    Configuration menu
    Copy the full SHA
    5a588ff View commit details
    Browse the repository at this point in the history