Cyclic Reference when trying to make a CodeBuild project to EKS

I’m trying to make two separate stack,

1. creates a bunch of CI/CD stuff, which includes a build stage to deploy to EKS cluster.
2. creates a EKS cluster and configures it.

When I tried to deploy the stacks, I get the following error. I guess it’s somewhat related to #5760 , #3300?

Reproduction Steps

1. CicdForAppStack

constructor(scope: cdk.Construct, id: string, props: CommonProps) {
    super(scope, id, props);
...
const deployToMainEKScluster = new codebuild.PipelineProject(this, 'deploy-to-main', {
            environment: {
                buildImage: codebuild.LinuxBuildImage.fromAsset(this, 'custom-image', {
                    directory: './buildimage'
                })
            },
            environmentVariables: { 
                'CLUSTER_NAME': {
                    value: `${props.cluster.clusterName}`
                  },
                'ECR_REPO_URI': {
                value: `${ecrForHelloPy.repositoryUri}`
              } 
            },
            buildSpec: codebuild.BuildSpec.fromObject({
                version: "0.2",
                phases: {
                  install: {
                    commands: [
                      'env',
                      'export TAG=${CODEBUILD_RESOLVED_SOURCE_VERSION}',
                      '/usr/local/bin/entrypoint.sh'                    ]
                  },
                  build: {
                    commands: [
                        `sed -i 's@CONTAINER_IMAGE@'"$ECR_REPO_URI:$TAG"'@' hello-py.yaml`,
                        'kubectl apply -f hello-py.yaml'
                    ]
                  }
                }})
        });

1. ClusterStack I think it’s quite ordinary stack for EKS, so I just add the link to the code, instead of pasting it over. https://github.com/yjw113080/aws-cdk-eks-multi-region/blob/master/lib/cluster-stack.ts

Error Log

/Users/jiwony/multi-cluster-ts/node_modules/@aws-cdk/core/lib/stack.ts:659
        throw new Error(`'${target.node.path}' depends on '${this.node.path}' (${cycle.join(', ')}). Adding this dependency (${reason}) would create a cyclic reference.`);
              ^
Error: 'CicdForAppStack' depends on 'v2-ClusterStack-ap-northeast-1' (CicdForAppStack -> v2-ClusterStack-ap-northeast-1/demogo-cluster/Resource/Resource/Default.Ref). Adding this dependency (v2-ClusterStack-ap-northeast-1 -> CicdForAppStack/deploy-to-main/Role/Resource.Arn) would create a cyclic reference.
    at ClusterStack._addAssemblyDependency (/Users/jiwony/multi-cluster-ts/node_modules/@aws-cdk/core/lib/stack.ts:659:15)
    at Object.addDependency (/Users/jiwony/multi-cluster-ts/node_modules/@aws-cdk/core/lib/deps.ts:44:20)
    at ClusterStack.addDependency (/Users/jiwony/multi-cluster-ts/node_modules/@aws-cdk/core/lib/stack.ts:364:5)
    at CicdForAppStack.prepareCrossReference (/Users/jiwony/multi-cluster-ts/node_modules/@aws-cdk/core/lib/stack.ts:926:24)
    at ClusterStack.prepare (/Users/jiwony/multi-cluster-ts/node_modules/@aws-cdk/core/lib/stack.ts:769:37)
    at ClusterStack.onPrepare (/Users/jiwony/multi-cluster-ts/node_modules/@aws-cdk/core/lib/construct-compat.ts:105:10)
    at Node.prepare (/Users/jiwony/multi-cluster-ts/node_modules/constructs/lib/construct.ts:441:28)
    at Node.synthesize (/Users/jiwony/multi-cluster-ts/node_modules/constructs/lib/construct.ts:399:10)
    at Function.synth (/Users/jiwony/multi-cluster-ts/node_modules/@aws-cdk/core/lib/construct-compat.ts:225:22)
    at App.synth (/Users/jiwony/multi-cluster-ts/node_modules/@aws-cdk/core/lib/app.ts:141:36)

Environment

• **CLI Version 😗*1.30.0 (build 4f54ff7
• **Framework Version:**1.30.0
• **OS 😗*OSX Mojave
• **Language 😗*Typescript

Other

---

This is 🐛 Bug Report

[skinny85]

I think I know this from the error message. Let me reformat it a little:

Error: 'CicdForAppStack' depends on 'v2-ClusterStack-ap-northeast-1':

CicdForAppStack -> v2-ClusterStack-ap-northeast-1/demogo-cluster/Resource/Resource/Default.Ref).

Adding this dependency: 

v2-ClusterStack-ap-northeast-1 -> CicdForAppStack/deploy-to-main/Role/Resource.Arn

would create a cyclic reference

I assume the CicdForAppStack/deploy-to-main/Role/Resource.Arn is because you need to add the role to the resource policy of the ECR repo to allow it access. Because of that, you can’t also have a dynamic reference to the Cluster - you have to hard-code it to avoid the dependency.

Does that make sense…?

[yjw113080]

Ah I got this. Thanks for your explanation! closing this issue. 😃