crossplane-contrib / provider-upjet-aws

Official AWS Provider for Crossplane by Upbound.
https://marketplace.upbound.io/providers/upbound/provider-aws
Apache License 2.0
137 stars 112 forks source link

lateinit version due to nodegroup conflict #1374

Closed stevendborrelli closed 1 week ago

stevendborrelli commented 1 week ago

Description of your changes

This is a followup to PR https://github.com/crossplane-contrib/provider-upjet-aws/pull/977, and adds the version field.

I've also changed one of the test subnets to AZ us-west-1c, as some accounts don't get access to 1a

aws ec2 describe-availability-zones --region us-west-1 
{
    "AvailabilityZones": [
        {
            "State": "available",
            "OptInStatus": "opt-in-not-required",
            "Messages": [],
            "RegionName": "us-west-1",
            "ZoneName": "us-west-1b",
            "ZoneId": "usw1-az3",
            "GroupName": "us-west-1",
            "NetworkBorderGroup": "us-west-1",
            "ZoneType": "availability-zone"
        },
        {
            "State": "available",
            "OptInStatus": "opt-in-not-required",
            "Messages": [],
            "RegionName": "us-west-1",
            "ZoneName": "us-west-1c",
            "ZoneId": "usw1-az1",
            "GroupName": "us-west-1",
            "NetworkBorderGroup": "us-west-1",
            "ZoneType": "availability-zone"
        }
    ]
}

How has this code been tested

Uptest run https://github.com/crossplane-contrib/provider-upjet-aws/actions/runs/9642201880/job/26589493440 on v1beta2 Cluster & NodeGroup

stevendborrelli commented 1 week ago

/test-examples="examples/eks/v1beta2/cluster.yaml,examples/eks/v1beta2/identityproviderconfig.yaml,examples/eks/v1beta2/nodegroup.yaml"