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

⚠️ Remove PortOpts.SecurityGroups #1291

Closed
wants to merge 2 commits into from
Closed

⚠️ Remove PortOpts.SecurityGroups #1291

wants to merge 2 commits into from

Commits on Mar 8, 2023

  1. ⚠ Remove PortOpts.SecurityGroupFilters 

    This turns PortOpts.SecurityGroups in v1alpha6 into
*[]SecurityGroupParam instead of *[]string and removes the
SecurityGroupFilters field which would contain the same information
otherwise.

Co-authored-by: Anwar Hassen <anwar.hassen@est.tech>
    @lentzi90
    lentzi90 and Anwar Hassen committed Mar 8, 2023
    Configuration menu
    Copy the full SHA
    1ef5ee1 View commit details
    Browse the repository at this point in the history

Commits on Mar 20, 2023

  1. Add conversion tests 

    This adds conversion tests especially for PortOpts. Both "normal" tests
and fuzzy tests are added for v1alpha6. Conversions and tests are updated also for
older API versions.
    @lentzi90
    lentzi90 committed Mar 20, 2023
    Configuration menu
    Copy the full SHA
    20cd37a View commit details
    Browse the repository at this point in the history