-
Notifications
You must be signed in to change notification settings - Fork 429
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
Add generate test duration file cron #2347
Add generate test duration file cron #2347
Conversation
Important Auto Review SkippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the WalkthroughThe updates introduce a new CI job that runs tests across different environments and generates a test duration report. It leverages a separate workflow to create the report, which includes setting up the environment, handling credentials, and installing dependencies. Upon detecting changes, it can also initiate a pull request. Changes
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
@coderabbitai review |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Review Status
Actionable comments generated: 0
Configuration used: .coderabbit.yaml
Files selected for processing (2)
- .github/workflows/ci-fast.yml (1 hunks)
- .github/workflows/generate-test-duration.yml (1 hunks)
Additional comments: 2
.github/workflows/ci-fast.yml (1)
- 89-101: The new job
test-generate-test-duration
has been added correctly with a matrix strategy for different operating systems, Python versions, and test environments. The job uses a custom action and inherits secrets as expected..github/workflows/generate-test-duration.yml (1)
- 1-220: The workflow
generate-test-duration.yml
is well-structured with clear inputs and a comprehensive set of jobs that cover various configurations and environments. It includes steps for setting up credentials, environment variables, and handling potential changes in the test duration file. The workflow also accounts for different operating systems and test environments, which is good for ensuring compatibility across various setups.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks ok, but this isn't a cron
job. It just runs every time we run the fast CI which feels maybe too often?
/fulltest |
…ithub.com:zenml-io/zenml into feature/OSSK-365-cron-test-duration-file-generate
Co-authored-by: Alex Strick van Linschoten <[email protected]>
* add generate test duration file cron * Refactor workflow to generate test duration file * Add test-generate-test-duration job to ci-fast workflow * Fix coverage script to handle test failures gracefully * Update secrets inheritance in ci-fast.yml * Add continue-on-error flag to Generate test duration step * Update .test_duration workflow * Update .test_duration file due to detected changes * Update test duration file with new changes * Update test duration file generation workflow * Remove unused job from ci-fast workflow * Add workflow_call trigger to generate-test-duration.yml * Remove tmate setup action from workflow * Update test duration file name * new changes on test duration file detected at 20240124-133657 * Update branch reference in generate-test-duration.yml * Remove unused test-generate-test-duration job * Update .github/workflows/generate-test-duration.yml Co-authored-by: Alex Strick van Linschoten <[email protected]> --------- Co-authored-by: ZenML GmbH <[email protected]> Co-authored-by: Alex Strick van Linschoten <[email protected]>
* add generate test duration file cron * Refactor workflow to generate test duration file * Add test-generate-test-duration job to ci-fast workflow * Fix coverage script to handle test failures gracefully * Update secrets inheritance in ci-fast.yml * Add continue-on-error flag to Generate test duration step * Update .test_duration workflow * Update .test_duration file due to detected changes * Update test duration file with new changes * Update test duration file generation workflow * Remove unused job from ci-fast workflow * Add workflow_call trigger to generate-test-duration.yml * Remove tmate setup action from workflow * Update test duration file name * new changes on test duration file detected at 20240124-133657 * Update branch reference in generate-test-duration.yml * Remove unused test-generate-test-duration job * Update .github/workflows/generate-test-duration.yml Co-authored-by: Alex Strick van Linschoten <[email protected]> --------- Co-authored-by: ZenML GmbH <[email protected]> Co-authored-by: Alex Strick van Linschoten <[email protected]>
Describe changes
I implemented/fixed _ to achieve _.
Pre-requisites
Please ensure you have done the following:
develop
and the open PR is targetingdevelop
. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.Types of changes
Summary by CodeRabbit