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

chore: mitigate intermitent failure of ci build #1333

Merged
merged 1 commit into from
May 22, 2024
Merged

Conversation

zxkane
Copy link
Contributor

@zxkane zxkane commented May 19, 2024


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

Summary

use the workaround suggested in nodejs/node#51555

Implementation highlights

(describe how the merge request does for feature changes, share the RFC link if it has)

Test checklist

  • add new test cases
  • all code changes are covered by unit tests
  • end-to-end tests
    • deploy web console with CloudFront + S3 + API gateway
    • deploy web console within VPC
    • deploy ingestion server
      • with MSK sink
      • with KDS sink
      • with S3 sink
    • deploy data processing
    • deploy data modeling
      • new Redshift Serverless
      • provisioned Redshift
      • Athena
    • deploy with reporting
    • streaming ingestion
      • with Redshift Serverless
      • with provisioned Redshift

Is it a breaking change

  • add parameters without default value in stack
  • introduce new service permission in stack
  • introduce new top level stack module

Miscellaneous

  • introduce new symbol link source file(s) to be shared among infra code, web console frontend, and web console backend

Copy link

Etl common library's test coverage

Overall Project 92.67% 🍏

There is no coverage information present for the Files changed

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Data Pipeline spark application's test coverage

Overall Project 98.14% 🍏

There is no coverage information present for the Files changed

@zxkane
Copy link
Contributor Author

zxkane commented May 19, 2024

SonarQube Quality Gate Result

  • Result: ✅ OK
  • Triggered by @zxkane on pull_request
Metric Status Value Error Threshold
Reliability rating ✅ OK 1 > 1
Security rating ✅ OK 1 > 1
Sqale rating ✅ OK 1 > 1
Coverage ✅ OK 83.60 < 80
Duplicated lines density ✅ OK 5.20 > 30
Blocker violations ✅ OK 0 > 0
Bugs ✅ OK 0 > 0
Code smells ✅ OK 8 > 40
Critical violations ✅ OK 0 > 0
Major violations ✅ OK 0 > 0
Vulnerabilities ✅ OK 0 > 0

View on SonarQube

updated: 5/19/2024, 05:26:43 (UTC+0)

@zxkane
Copy link
Contributor Author

zxkane commented May 19, 2024

SonarQube Code Analytics

Quality Gate passed

passed

Additional information

The following metrics might not affect the Quality Gate status but improving them will improve your project code quality.

Issues

bug 0 Bugs

vulnerability 0 Vulnerabilities

code_smell 0 Code Smells

Coverage and Duplications

No data Coverage

No data Duplication

Copy link

Etl common library's test coverage

Overall Project 92.67% 🍏

There is no coverage information present for the Files changed

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Data Pipeline spark application's test coverage

Overall Project 98.14% 🍏

There is no coverage information present for the Files changed

@zxkane
Copy link
Contributor Author

zxkane commented May 19, 2024

SonarQube Quality Gate Result

  • Result: ✅ OK
  • Triggered by @zxkane on pull_request
Metric Status Value Error Threshold
Reliability rating ✅ OK 1 > 1
Security rating ✅ OK 1 > 1
Sqale rating ✅ OK 1 > 1
Coverage ✅ OK 83.60 < 80
Duplicated lines density ✅ OK 5.20 > 30
Blocker violations ✅ OK 0 > 0
Bugs ✅ OK 0 > 0
Code smells ✅ OK 8 > 40
Critical violations ✅ OK 0 > 0
Major violations ✅ OK 0 > 0
Vulnerabilities ✅ OK 0 > 0

View on SonarQube

updated: 5/19/2024, 07:10:26 (UTC+0)

Copy link

Etl common library's test coverage

Overall Project 92.67% 🍏

There is no coverage information present for the Files changed

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Data Pipeline spark application's test coverage

Overall Project 98.14% 🍏

There is no coverage information present for the Files changed

@zxkane
Copy link
Contributor Author

zxkane commented May 19, 2024

SonarQube Quality Gate Result

  • Result: ✅ OK
  • Triggered by @zxkane on pull_request
Metric Status Value Error Threshold
Reliability rating ✅ OK 1 > 1
Security rating ✅ OK 1 > 1
Sqale rating ✅ OK 1 > 1
Coverage ✅ OK 83.60 < 80
Duplicated lines density ✅ OK 5.20 > 30
Blocker violations ✅ OK 0 > 0
Bugs ✅ OK 0 > 0
Code smells ✅ OK 8 > 40
Critical violations ✅ OK 0 > 0
Major violations ✅ OK 0 > 0
Vulnerabilities ✅ OK 0 > 0

View on SonarQube

updated: 5/19/2024, 15:42:56 (UTC+0)

Copy link

Etl common library's test coverage

Overall Project 92.67% 🍏

There is no coverage information present for the Files changed

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Data Pipeline spark application's test coverage

Overall Project 98.14% 🍏

There is no coverage information present for the Files changed

Copy link

Etl common library's test coverage

Overall Project 92.67% 🍏

There is no coverage information present for the Files changed

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Data Pipeline spark application's test coverage

Overall Project 98.14% 🍏

There is no coverage information present for the Files changed

@zxkane
Copy link
Contributor Author

zxkane commented May 20, 2024

SonarQube Quality Gate Result

  • Result: ✅ OK
  • Triggered by @zxkane on pull_request
Metric Status Value Error Threshold
Reliability rating ✅ OK 1 > 1
Security rating ✅ OK 1 > 1
Sqale rating ✅ OK 1 > 1
Coverage ✅ OK 83.60 < 80
Duplicated lines density ✅ OK 5.20 > 30
Blocker violations ✅ OK 0 > 0
Bugs ✅ OK 0 > 0
Code smells ✅ OK 8 > 40
Critical violations ✅ OK 0 > 0
Major violations ✅ OK 0 > 0
Vulnerabilities ✅ OK 0 > 0

View on SonarQube

updated: 5/20/2024, 07:30:24 (UTC+0)

@zxkane zxkane changed the title chore: mitigate ci build intermitent failure chore: mitigate intermitent failure of ci build May 20, 2024
Copy link

Etl common library's test coverage

Overall Project 92.67% 🍏

There is no coverage information present for the Files changed

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Data Pipeline spark application's test coverage

Overall Project 98.14% 🍏

There is no coverage information present for the Files changed

@zxkane
Copy link
Contributor Author

zxkane commented May 20, 2024

SonarQube Quality Gate Result

  • Result: ✅ OK
  • Triggered by @zxkane on pull_request
Metric Status Value Error Threshold
Reliability rating ✅ OK 1 > 1
Security rating ✅ OK 1 > 1
Sqale rating ✅ OK 1 > 1
Coverage ✅ OK 83.60 < 80
Duplicated lines density ✅ OK 5.20 > 30
Blocker violations ✅ OK 0 > 0
Bugs ✅ OK 0 > 0
Code smells ✅ OK 8 > 40
Critical violations ✅ OK 0 > 0
Major violations ✅ OK 0 > 0
Vulnerabilities ✅ OK 0 > 0

View on SonarQube

updated: 5/20/2024, 09:29:36 (UTC+0)

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Data Pipeline spark application's test coverage

Overall Project 98.15% 🍏

There is no coverage information present for the Files changed

@zxkane
Copy link
Contributor Author

zxkane commented May 21, 2024

SonarQube Quality Gate Result

  • Result: ✅ OK
  • Triggered by @zxkane on pull_request
Metric Status Value Error Threshold
Reliability rating ✅ OK 1 > 1
Security rating ✅ OK 1 > 1
Sqale rating ✅ OK 1 > 1
Coverage ✅ OK 83.50 < 80
Duplicated lines density ✅ OK 5.20 > 30
Blocker violations ✅ OK 0 > 0
Bugs ✅ OK 0 > 0
Code smells ✅ OK 8 > 40
Critical violations ✅ OK 0 > 0
Major violations ✅ OK 0 > 0
Vulnerabilities ✅ OK 0 > 0

View on SonarQube

updated: 5/21/2024, 06:17:16 (UTC+0)

Copy link

Etl common library's test coverage

Overall Project 92.67% 🍏

There is no coverage information present for the Files changed

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Data Pipeline spark application's test coverage

Overall Project 98.15% 🍏

There is no coverage information present for the Files changed

@zxkane
Copy link
Contributor Author

zxkane commented May 21, 2024

SonarQube Quality Gate Result

  • Result: ✅ OK
  • Triggered by @zxkane on pull_request
Metric Status Value Error Threshold
Reliability rating ✅ OK 1 > 1
Security rating ✅ OK 1 > 1
Sqale rating ✅ OK 1 > 1
Coverage ✅ OK 83.60 < 80
Duplicated lines density ✅ OK 5.20 > 30
Blocker violations ✅ OK 0 > 0
Bugs ✅ OK 0 > 0
Code smells ✅ OK 8 > 40
Critical violations ✅ OK 0 > 0
Major violations ✅ OK 0 > 0
Vulnerabilities ✅ OK 0 > 0

View on SonarQube

updated: 5/21/2024, 15:03:51 (UTC+0)

@zxkane zxkane requested review from tyyzqmf and chenhaiyun May 22, 2024 05:47
@zxkane zxkane marked this pull request as ready for review May 22, 2024 05:48
Copy link

Etl common library's test coverage

Overall Project 92.67% 🍏

There is no coverage information present for the Files changed

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Etl common library's test coverage

Overall Project 92.67% 🍏

There is no coverage information present for the Files changed

Copy link

Streaming ingestion flink application's test coverage

Overall Project 94.57% 🍏

There is no coverage information present for the Files changed

Copy link

Data Pipeline spark application's test coverage

Overall Project 98.15% 🍏

There is no coverage information present for the Files changed

@tyyzqmf tyyzqmf enabled auto-merge May 22, 2024 06:10
Copy link

Data Pipeline spark application's test coverage

Overall Project 98.15% 🍏

There is no coverage information present for the Files changed

Copy link

Code Coverage

Package Line Rate Branch Rate Health
src 100% 82%
src.analytics 100% 100%
src.analytics.lambdas 92% 89%
src.analytics.lambdas.clear-expired-events-workflow 100% 100%
src.analytics.lambdas.custom-resource 96% 87%
src.analytics.lambdas.load-data-workflow 99% 90%
src.analytics.lambdas.refresh-materialized-views-workflow 96% 85%
src.analytics.lambdas.scan-metadata-workflow 97% 88%
src.analytics.lambdas.sql-execution-sfn 93% 60%
src.analytics.lambdas.user-segments-workflow 96% 77%
src.analytics.private 100% 92%
src.analytics.private.athena 100% 100%
src.analytics.private.segments 100% 100%
src.base-lib.src 100% 100%
src.base-lib.src.auth 89% 65%
src.base-lib.src.common 97% 72%
src.base-lib.src.constant 100% 100%
src.base-lib.src.model 100% 100%
src.common 98% 80%
src.common.lambda 97% 86%
src.control-plane 99% 71%
src.control-plane.auth 93% 12%
src.control-plane.backend 100% 84%
src.control-plane.backend.lambda.api 97% 33%
src.control-plane.backend.lambda.api.common 92% 79%
src.control-plane.backend.lambda.api.middle-ware 88% 79%
src.control-plane.backend.lambda.api.model 96% 56%
src.control-plane.backend.lambda.api.router 98% 97%
src.control-plane.backend.lambda.api.service 85% 67%
src.control-plane.backend.lambda.api.service.quicksight 93% 81%
src.control-plane.backend.lambda.api.service.segments 95% 76%
src.control-plane.backend.lambda.api.store.aws 87% 56%
src.control-plane.backend.lambda.api.store.dynamodb 90% 80%
src.control-plane.backend.lambda.api.test.api 99% 81%
src.control-plane.backend.lambda.api.test.api.segments 100% 100%
src.control-plane.backend.lambda.batch-insert-ddb 94% 86%
src.control-plane.backend.lambda.listen-stack-status 81% 49%
src.control-plane.backend.lambda.listen-state-status 91% 50%
src.control-plane.backend.lambda.sfn-action 88% 50%
src.control-plane.backend.lambda.sfn-workflow 80% 67%
src.control-plane.backend.layer.lambda-web-adapter 98% 40%
src.control-plane.private 100% 100%
src.data-pipeline 100% 100%
src.data-pipeline.lambda.copy-assets 97% 90%
src.data-pipeline.lambda.emr-job-state-listener 96% 91%
src.data-pipeline.lambda.emr-job-submitter 100% 78%
src.data-pipeline.lambda.emr-serverless-app 96% 86%
src.data-pipeline.lambda.partition-syncer 98% 89%
src.data-pipeline.tables 100% 100%
src.data-pipeline.utils 100% 100%
src.ingestion-server.custom-resource 100% 100%
src.ingestion-server.custom-resource.delete-ecs-cluster 98% 90%
src.ingestion-server.custom-resource.update-alb-rules 100% 78%
src.ingestion-server.kafka-s3-connector 100% 100%
src.ingestion-server.kafka-s3-connector.custom-resource.kafka-s3-sink-connector 96% 82%
src.ingestion-server.kinesis-data-stream 100% 100%
src.ingestion-server.kinesis-data-stream.kinesis-to-s3-lambda 100% 100%
src.ingestion-server.kinesis-data-stream.private 100% 100%
src.ingestion-server.server 98% 89%
src.ingestion-server.server-v2 100% 79%
src.ingestion-server.server-v2.private 100% 82%
src.ingestion-server.server.private 100% 89%
src.metrics 100% 100%
src.metrics.custom-resource.add-sns-subscription 95% 86%
src.metrics.custom-resource.get-interval 99% 97%
src.metrics.custom-resource.put-dashboard 97% 84%
src.metrics.custom-resource.set-metrics-widgets 98% 93%
src.private 96% 73%
src.reporting 100% 80%
src.reporting.lambda.custom-resource.quicksight 94% 81%
src.reporting.private 90% 54%
src.streaming-ingestion 100% 100%
src.streaming-ingestion.lambdas.custom-resource 86% 63%
src.streaming-ingestion.private 99% 89%
src.streaming-ingestion.redshift 100% 100%
test 96% 83%
test.common 93% 100%
test.control-plane 97% 64%
test.ingestion-server.server 97% 88%
Summary 96% (75521 / 78629) 74% (6377 / 8565)

Minimum allowed line rate is 60%

@tyyzqmf tyyzqmf added this pull request to the merge queue May 22, 2024
@zxkane
Copy link
Contributor Author

zxkane commented May 22, 2024

SonarQube Quality Gate Result

  • Result: ✅ OK
  • Triggered by @zxkane on pull_request
Metric Status Value Error Threshold
Reliability rating ✅ OK 1 > 1
Security rating ✅ OK 1 > 1
Sqale rating ✅ OK 1 > 1
Coverage ✅ OK 83.80 < 80
Duplicated lines density ✅ OK 5.20 > 30
Blocker violations ✅ OK 0 > 0
Bugs ✅ OK 0 > 0
Code smells ✅ OK 8 > 40
Critical violations ✅ OK 0 > 0
Major violations ✅ OK 0 > 0
Vulnerabilities ✅ OK 0 > 0

View on SonarQube

updated: 5/22/2024, 06:24:10 (UTC+0)

Merged via the queue into main with commit cdcc60b May 22, 2024
18 checks passed
@tyyzqmf tyyzqmf deleted the local-esbuild branch May 22, 2024 06:43
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

Successfully merging this pull request may close these issues.

3 participants