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.

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

Use Terraform Plugin SDK to Reconcile MRs #938

Merged
merged 106 commits into from
Nov 16, 2023

Conversation

ulucinar
Copy link
Collaborator

@ulucinar ulucinar commented Oct 26, 2023

Description of your changes

This PR employs the controller.noForkExternal & controller.noForkAsyncExternal external clients from upjet to reconcile all but the following MRs to reconcile without forking any Terraform CLI or Terraform provider processes:

  • SecurityGroupIngressRule.ec2
  • SecurityGroupEgressRule.ec2
  • Environment.appconfig
  • UserPoolClient.cognitoidp
  • Domain.simpledb

The above resources are reconciled using the legacy Terraform CLI-based external clients.

Some more details on these clients can be found here.

I have:

  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

The following resources have been validated either by manual tests or uptests.

@ulucinar ulucinar marked this pull request as draft October 26, 2023 16:12
@ulucinar
Copy link
Collaborator Author

/test-examples="examples/ec2/vpc.yaml"

@ulucinar
Copy link
Collaborator Author

/test-examples="examples/rds/instance.yaml"

@ulucinar
Copy link
Collaborator Author

/test-examples="examples/elasticache/replicationgroup.yaml"

1 similar comment
@sergenyalcin
Copy link
Collaborator

/test-examples="examples/elasticache/replicationgroup.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/eks/nodegroup.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/ec2/serialconsoleaccess.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/ec2/mainroutetableassociation.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/ec2/spotinstancerequest.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/appautoscaling/policy.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/directconnect/bgppeer.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/cloudfront/distribution.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/cloudwatchlogs/metricfilter.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/glue/datacatalogencryptionsettings.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/glue/catalogtable.yaml,examples/glue/catalogdatabase.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/directconnect/bgppeer.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/kms/grant.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/glue/catalogtable.yaml,examples/glue/catalogdatabase.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/mq/broker.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/glue/catalogtable.yaml,examples/glue/catalogdatabase.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/networkfirewall/firewall.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/sns/smspreferences.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/mq/broker.yaml"

1 similar comment
@sergenyalcin
Copy link
Collaborator

/test-examples="examples/mq/broker.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/ec2/flowlog.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/apigatewayv2/api-http.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/directconnect/gatewayassociation.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/ec2/vpcendpointsecuritygroupassociation.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/ec2/flowlog.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/s3/bucketlifecycleconfiguration.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/rds/clusterparametergroup.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/elbv2/lblistener.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/timestreamwrite/database.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/directconnect/bgppeer.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/s3/bucketlogging.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/kms/key.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/s3/bucketpublicaccessblock.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/iam/role.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/ec2/vpcendpointsecuritygroupassociation.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/s3/bucketserversideencryptionconfiguration.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/ec2/securitygrouprule-self-true.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/eks/addon.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/directconnect/gatewayassociation.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/s3/bucketversioning.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/rds/subnetgroup.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/kafka/cluster.yaml"

@sergenyalcin
Copy link
Collaborator

/test-examples="examples/directconnect/transitvirtualinterface.yaml"

… argument from 10 to 100

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
@ulucinar
Copy link
Collaborator Author

/test-examples="examples/ec2/vpc.yaml"

Signed-off-by: Alper Rifat Ulucinar <ulucinar@users.noreply.github.com>
@ulucinar
Copy link
Collaborator Author

/test-examples="examples/ec2/vpc.yaml"

@ulucinar
Copy link
Collaborator Author

make lint runs successfully on our local development machines. Looks like this is an issue with the runners we are using. Please refer to the issue mentioned in the description of this PR for further details. We tried switching to Ubuntu 20.04 on smaller machines but the linter job is now failing to due high memory consumption.

Copy link
Collaborator

@sergenyalcin sergenyalcin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! LGTM!

@ulucinar ulucinar merged commit 3000dfd into crossplane-contrib:main Nov 16, 2023
9 of 10 checks passed
@ulucinar ulucinar deleted the no-fork-v4.67.0 branch November 16, 2023 12:48
@turkenf turkenf mentioned this pull request Nov 20, 2023
1 task
@mergenci mergenci mentioned this pull request Feb 26, 2024
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants