Skip to content
This repository was archived by the owner on Aug 7, 2025. It is now read-only.

use new artifacts (pro) for Docs Update Workflow#516

Merged
steffyP merged 3 commits intomainfrom
artifact-retrieval
Mar 27, 2023
Merged

use new artifacts (pro) for Docs Update Workflow#516
steffyP merged 3 commits intomainfrom
artifact-retrieval

Conversation

@steffyP
Copy link
Member

@steffyP steffyP commented Mar 15, 2023

When moving to the new pro pipeline, some of the artifacts required for updating the docs need to be retrieved from different locations:

Coverage information for pro (capture not-implemented)

  • previously run in CircleCI, with the move only community details are available in CircleCI
  • added new job in the workflow to retrieve the artifact

Metrics Raw Data for pro (test details)

  • previously already retrieved from GitHub, nothing should change here

In this PR:

  • updated the workflow
  • updated the scripts to retrieve the resources from CircleCI + Github

Open TODOs:

  • adapt workflow + artifact names + repository name
  • remove the TODO (rm -rf pro) from get_latest_circleci_metrics.sh (currently the pro-artifacts are still available)

/cc @alexrashed

@github-actions
Copy link

github-actions bot commented Mar 15, 2023

🎊 PR Preview has been successfully built and deployed to https://localstack-docs-preview-pr-516.surge.sh 🎊

Base automatically changed from coverage-docs-v2 to main March 20, 2023 13:38
@steffyP steffyP force-pushed the artifact-retrieval branch from 175e31e to edab814 Compare March 27, 2023 08:09
@steffyP
Copy link
Member Author

steffyP commented Mar 27, 2023

latest workflow update tested here: #534

@steffyP steffyP marked this pull request as ready for review March 27, 2023 08:17
@steffyP steffyP requested a review from alexrashed March 27, 2023 08:17
Copy link
Member

@alexrashed alexrashed left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adjusting to the new pipelines so quickly! This looks great! 🚀

@steffyP steffyP merged commit 35926ab into main Mar 27, 2023
@steffyP steffyP deleted the artifact-retrieval branch March 27, 2023 09:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants