Skip to content

*: kurtosis smoke script fix#4431

Merged
pinebit merged 2 commits intomainfrom
pinebit/kurtosis-test-fix
Mar 31, 2026
Merged

*: kurtosis smoke script fix#4431
pinebit merged 2 commits intomainfrom
pinebit/kurtosis-test-fix

Conversation

@pinebit
Copy link
Copy Markdown
Collaborator

@pinebit pinebit commented Mar 31, 2026

Fixed wrong expression.

category: fixbuild
ticket: none

@pinebit pinebit requested review from KaloyanTanev and Copilot March 31, 2026 15:07
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Kurtosis smoke test GitHub Actions workflow to fix a previously incorrect/undesired job timeout configuration.

Changes:

  • Replaced the job timeout-minutes expression with a fixed value (540).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pinebit pinebit disabled auto-merge March 31, 2026 15:10
@pinebit pinebit requested a review from Copilot March 31, 2026 15:11
@pinebit pinebit enabled auto-merge (squash) March 31, 2026 15:11
@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@pinebit pinebit merged commit e5ffcee into main Mar 31, 2026
14 of 15 checks passed
@pinebit pinebit deleted the pinebit/kurtosis-test-fix branch March 31, 2026 15:22
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.44%. Comparing base (c89a7ee) to head (80daa6a).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4431      +/-   ##
==========================================
- Coverage   56.47%   56.44%   -0.03%     
==========================================
  Files         244      244              
  Lines       32550    32550              
==========================================
- Hits        18381    18372       -9     
- Misses      11820    11826       +6     
- Partials     2349     2352       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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