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

upbound aws provider fails to create/delete SNS topic on china due to bad aws partition (shall be aws-cn and not aws) #881

Closed
yebolenko opened this issue Sep 14, 2023 · 15 comments
Labels
bug Something isn't working is:triaged Indicates that an issue has been reviewed. stale

Comments

@yebolenko
Copy link

What happened?

How can we reproduce it?

What environment did it happen in?

  • Crossplane Version: docker.io/crossplane/crossplane:v1.13.2
  • Provider Version: xpkg.upbound.io/upbound/provider-aws:v0.38.0
  • Kubernetes Version: 1.27
  • Kubernetes Distribution: EKS
@yebolenko yebolenko added bug Something isn't working needs:triage labels Sep 14, 2023
@yebolenko
Copy link
Author

Please ensure that all upbound components use aws partition as a parameter and not hardcoded aws

@yebolenko
Copy link
Author

yebolenko commented Sep 14, 2023

it seems that some info in the issue has not been update:
I'm summarizing it again:
We are trying to create/delete SNS Topic with crossplane/upbound on aws China region cn-north-1
When SNS topic shall be created/deleted, it fails with the following error:
InvalidParameter: Invalid parameter: TopicArn Reason: A REDACTED ARN must begin with arn:aws-cn, not arn:aws:sns:REDACTED:REDACTED:ye-sns-topic-2-demo-china-cnn1\n\tstatus code: 40I

@yebolenko yebolenko changed the title upbound aws provider fails to delete SNS topic on china due to bad partition (shall be aws-cn and not aws) upbound aws provider fails to create/delete SNS topic on china due to bad aws partition (shall be aws-cn and not aws) Sep 14, 2023
@yebolenko
Copy link
Author

any progress?

@turkenf
Copy link
Collaborator

turkenf commented Oct 11, 2023

@yebolenko, could you please fill in the description properly? Reproduction steps, the MRs you used, expected and actual behavior.

@yebolenko
Copy link
Author

It's always reproducible. You just need to create/delete SNS topic on aws region cn-north-1 (china).
It doesn't work due to hardcoded aws partition id (aws) and not aws-cn.
I believe it's general issue also in many other resources. You need to add ability to pass aws partition id as parameter (aws is default)

@yebolenko
Copy link
Author

any update? it's URGENT issue for us. We cannot use SNS and (maybe other components) on china.
As I said, we use an official upbound release

@turkenf
Copy link
Collaborator

turkenf commented Nov 19, 2023

Please provide the requested information in the description.

@yebolenko
Copy link
Author

Providing again all details:

What happened? When SNS topic shall be created/deleted, it fails with the following error:
InvalidParameter: Invalid parameter: TopicArn Reason: A REDACTED ARN must begin with arn:aws-cn, not arn:aws:sns:REDACTED:REDACTED:ye-sns-topic-2-demo-china-cnn1\n\tstatus code: 40I
How can we reproduce it? It's always reproducible. You just need to create/delete SNS topic on aws region cn-north-1 (china).
What environment did it happen in?
Crossplane Version: docker.io/crossplane/crossplane:v1.13.2
Provider Version: xpkg.upbound.io/upbound/provider-aws:v0.38.0
Kubernetes Version: 1.27
Kubernetes Distribution: EKS
Notes:
It doesn't work due to hardcoded aws partition id (aws) and not aws-cn.
I believe it's general issue also in many other resources. You need to add ability to pass aws partition id as parameter (aws is default)

@yebolenko
Copy link
Author

As I said above, it seems that we already have similar issue (with the same root cause, that hardcoded aws partition is used) for GOV
#757

@turkenf
Copy link
Collaborator

turkenf commented Nov 21, 2023

Hi @yebolenko,

Thank you for raising this issue, it can be reproducible with the following MR:

apiVersion: sns.aws.upbound.io/v1beta1
kind: Topic
metadata:
  name: example
  labels:
    testing.upbound.io/example-name: sns
spec:
  forProvider:
    region: cn-north-1

This issue will be sorted and handled according to the prioritization policy.

@turkenf turkenf added is:triaged Indicates that an issue has been reviewed. and removed needs:information needs:triage labels Nov 21, 2023
@yebolenko
Copy link
Author

Fatih, thank you. Please let me know when it will be resolved. As I said, it's not only SNS. It's a general issue with hardcoded aws partition Id (aws:) and I believe that some global fix is required (which will fix in general special regions such as China and AWS GOV)

Copy link

github-actions bot commented Apr 2, 2024

This provider repo does not have enough maintainers to address every issue. Since there has been no activity in the last 90 days it is now marked as stale. It will be closed in 14 days if no further activity occurs. Leaving a comment starting with /fresh will mark this issue as not stale.

Copy link

This provider repo does not have enough maintainers to address every issue. Since there has been no activity in the last 90 days it is now marked as stale. It will be closed in 14 days if no further activity occurs. Leaving a comment starting with /fresh will mark this issue as not stale.

@github-actions github-actions bot added the stale label Aug 11, 2024
Copy link

This issue is being closed since there has been no activity for 14 days since marking it as stale. If you still need help, feel free to comment or reopen the issue!

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Aug 27, 2024
@cnshadzh
Copy link

cnshadzh commented Oct 9, 2024

The issue still exists, failed to create sns topic in region cn-northwest-1 with the same error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working is:triaged Indicates that an issue has been reviewed. stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants