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

CFN Template cid-cfn.yml doesn't account for ResourcePrefix #1045

Open
pauldtill opened this issue Nov 26, 2024 · 3 comments
Open

CFN Template cid-cfn.yml doesn't account for ResourcePrefix #1045

pauldtill opened this issue Nov 26, 2024 · 3 comments

Comments

@pauldtill
Copy link

The aggregation template allows the customer to specify the parameter ResourcePrefix - which we used to avoid some collisions with other deployments.

Unfortunately the main dashboards template doesn't account for this in the bucket name (at least - maybe other places). Here for example the bucket is hard coded with a cid- prefix -

                Resource:
                  - !Sub arn:${AWS::Partition}:s3:::cid-${AWS::AccountId}-data-exports # prefix for data-exports hardcoded here

For the old 1.0 configuration the bucket name could be passed in as a parameter to handle this difference in naming, but with the above this isn't possible without directly changing the template.

Other resources in the initial aggregator template also use this resource prefix for naming e.g. CIDDatabase (AWS::Glue::Database) so I'm also concerned they won't be picked up correctly if the 2nd template isn't aware of the prefix used.

Apologies if I missed something.

@iakov-aws
Copy link
Collaborator

Hi @pauldtill
I confirm that ResourcePrefix is not fully functional. This is mainly linked to the fact that the python code does not support deployment of multiple environments (dashboards and datasets prefixes, neither athena tables and databases) so this parameter currently cannot be used as of today.

Normally you can specify a bucket prefix in dataexports stack, it will create a policy DataExportsReadAccess That then will be attached runtime by the lambda of dashboard creation or you can attach this policy to the QucikSight Role manually. The hardcoded value you point to are here only for backward compatibility.

@pauldtill
Copy link
Author

@iakov-aws - this is unfortunate, we already have the source accounts (40+) setup using this resource prefix and replicating their data centrally, also using resource prefix AND we have 3 years worth of data backfilled. We also already have the dashboards deployed but using CUR 1.0, and need to update them to 2.0 - the prefix worked fine with the 1.0 setup.

Is there no way I can get the solution to work now with what has already been deployed i.e. do I need to start again from scratch ?

Perhaps this parameter should be completely removed, or at least moved to the "do not use" section to make this limitation clear?

@iakov-aws
Copy link
Collaborator

Hello Paul,

Can you contact your technical account manager from AWS to set up a call with one of cid team members?

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

No branches or pull requests

2 participants