Merge pull request #2003 from haooliveira84/sq-region

feat(ec2/securitygroup)!: Make region a required field

apis/ec2/v1beta1/securitygroup_types.go

@@ -24,13 +24,9 @@ import (
 // SecurityGroupParameters define the desired state of an AWS VPC Security
 // Group.
 type SecurityGroupParameters struct {
-	// TODO(muvaf): Region is a required field but in order to keep backward compatibility
-	// with old Provider type and not bear the cost of bumping to v1beta2, we're
-	// keeping it optional for now. Reconsider before v1beta2 or v1.
 
 	// Region is the region you'd like your SecurityGroup to be created in.
-	// +optional
-	Region *string `json:"region,omitempty"`
+	Region string `json:"region"`
 
 	// A description of the security group.
 	// +immutable

package/crds/ec2.aws.crossplane.io_securitygroups.yaml

@@ -795,6 +795,7 @@ spec:
                 required:
                 - description
                 - groupName
+                - region
                 type: object
               managementPolicies:
                 default:

pkg/controller/ec2/securitygroup/controller.go

@@ -113,7 +113,7 @@ func (c *connector) Connect(ctx context.Context, mg resource.Managed) (managed.E
 	if !ok {
 		return nil, errors.New(errUnexpectedObject)
 	}
-	cfg, err := connectaws.GetConfig(ctx, c.kube, mg, aws.ToString(cr.Spec.ForProvider.Region))
+	cfg, err := connectaws.GetConfig(ctx, c.kube, mg, cr.Spec.ForProvider.Region)
 	if err != nil {
 		return nil, err
 	}