Merge pull request #645 from aws-samples/doc/poc-guidance

poc guidance improvement and minor fix

Author: lmouhib

poc/cdk.json

@@ -35,7 +35,7 @@
     "@aws-cdk/aws-iam:standardizedServicePrincipals": true,
     "@aws-cdk/aws-ecs:disableExplicitDeploymentControllerForCircuitBreaker": true,
     "@aws-cdk/aws-iam:importedRoleStackSafeDefaultPolicyName": true,
-    "@aws-cdk/aws-s3:serverAccessLogsUseBucketPolicy": true,
+    "@aws-cdk/aws-s3:serverAccessLogsUseBucketPolicy": false,
     "@aws-cdk/aws-route53-patters:useCertificate": true,
     "@aws-cdk/customresources:installLatestAwsSdkDefault": false,
     "@aws-cdk/aws-rds:databaseProxyUniqueResourceName": true,

poc/poc/poc_stack.py

@@ -9,8 +9,8 @@
     KubernetesVersion,
     Cluster
     )
-from aws_cdk.aws_prometheus import(
-    PrometheusWorkspace
+from aws_cdk.aws_aps import(
+    CfnWorkspace
 )
 import aws_cdk.aws_iam as iam
 
@@ -21,7 +21,7 @@ def __init__(self, scope: Construct, construct_id: str, **kwargs) -> None:
 
         kubectl = KubectlV22Layer (self, 'kubectl_layer')
 
-        prometheus_workspace = PrometheusWorkspace(self, 'PrometheusWorkspace')
+        prometheus_workspace = CfnWorkspace(self, 'PrometheusWorkspace')
 
         emr_eks = ara.EmrEksCluster.get_or_create(self,
                                                   eks_cluster_name='poc-cluster',
@@ -66,4 +66,4 @@ def __init__(self, scope: Construct, construct_id: str, **kwargs) -> None:
         CfnOutput(self, 'VirtualClusterId', value=my_vc.attr_id)
         CfnOutput(self, 'RoleArn', value=role.role_arn)
         CfnOutput(self, 'CriticalConfig', value=emr_eks.critical_default_config)
-        CfnOutput(self, 'Prometheus', value=prometheus_workspace.attr_id)
+        CfnOutput(self, 'Prometheus', value=prometheus_workspace.attr_arn)