-
Notifications
You must be signed in to change notification settings - Fork 267
ec2 transitgateway code-gen problem zz_conversions.go #957
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
Comments
Issues go stale after 90d of inactivity. |
@haarchri Are you still experiencing the same issue? i see that https://github.com/haarchri/provider-aws/blob/ec2-transitgateway/pkg/controller/ec2/transitgateway/zz_conversions.go#L209-L272 got the correct field names |
@a-hilaly we skip the fields in generator-config - in general i think we have a problem with 4 fields: |
/remove-lifecycle stale |
Issues go stale after 90d of inactivity. |
@haarchri are you still experiencing this issue? |
Stale issues rot after 30d of inactivity. |
Please reopen if this still blocks you |
faced some problem in ec2 transitgateway during code-gen in crossplane project crossplane-contrib/provider-aws#831 in func GenerateModifyTransitGatewayInput
https://github.com/haarchri/provider-aws/blob/ec2-transitgateway/pkg/controller/ec2/transitgateway/zz_conversions.go#L209-L272
wrong:
cr.Spec.ForProvider.Options.AddTransitGatewayCIDRBlocks
cr.Spec.ForProvider.Options.AssociationDefaultRouteTableID
cr.Spec.ForProvider.Options.PropagationDefaultRouteTableID
cr.Spec.ForProvider.Options.RemoveTransitGatewayCIDRBlocks
correct:
cr.Spec.ForProvider.Options.TransitGatewayCIDRBlocks
cr.Spec.ForProvider.Options.DefaultRouteTableAssociation
cr.Spec.ForProvider.Options.DefaultRouteTablePropagation
cr.Spec.ForProvider.Options.TransitGatewayCIDRBlocks
crossplane-contrib/provider-aws#831
#489
The text was updated successfully, but these errors were encountered: