You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iggy.apache.org by "hubcio (via GitHub)" <gi...@apache.org> on 2026/03/20 16:44:59 UTC

[PR] fix(ci): prevent flaky keyring tests by deferring unlock to test phase [iggy]

hubcio opened a new pull request, #2995:
URL: https://github.com/apache/iggy/pull/2995

   gnome-keyring collection auto-locks after idle timeout.
   Unlocking before `cargo build` left a 7+ minute idle
   window (build + compile) before tests ran, causing
   org.freedesktop.Secret.Error.IsLocked ~10% of CI runs.
   
   Move D-Bus and keyring setup to right before test
   execution in both pre-merge and post-merge workflows.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix(ci): prevent flaky keyring tests by deferring unlock to test phase [iggy]

Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #2995:
URL: https://github.com/apache/iggy/pull/2995#issuecomment-4099517103

   ## [Codecov](https://app.codecov.io/gh/apache/iggy/pull/2995?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   :white_check_mark: All modified and coverable lines are covered by tests.
   :white_check_mark: Project coverage is 70.54%. Comparing base ([`f0e8578`](https://app.codecov.io/gh/apache/iggy/commit/f0e85781d75ac4967cdc5b71e3d101068299fc66?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`3f221bd`](https://app.codecov.io/gh/apache/iggy/commit/3f221bdb29f54e941c5ef3a4cc1b3f9fcf048bd2?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
   :warning: Report is 1 commits behind head on master.
   
   <details><summary>Additional details and impacted files</summary>
   
   
   
   ```diff
   @@             Coverage Diff              @@
   ##             master    #2995      +/-   ##
   ============================================
   - Coverage     71.81%   70.54%   -1.27%     
     Complexity      930      930              
   ============================================
     Files          1116     1053      -63     
     Lines         92616    87809    -4807     
     Branches      70139    70049      -90     
   ============================================
   - Hits          66512    61948    -4564     
   + Misses        23543    23298     -245     
   - Partials       2561     2563       +2     
   ```
   
   | [Flag](https://app.codecov.io/gh/apache/iggy/pull/2995/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | |
   |---|---|---|
   | [node](https://app.codecov.io/gh/apache/iggy/pull/2995/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `87.29% <ø> (-4.23%)` | :arrow_down: |
   
   Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#carryforward-flags-in-the-pull-request-comment) to find out more.
   [see 81 files with indirect coverage changes](https://app.codecov.io/gh/apache/iggy/pull/2995/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)
   </details>
   <details><summary> :rocket: New features to boost your workflow: </summary>
   
   - :snowflake: [Test Analytics](https://docs.codecov.com/docs/test-analytics): Detect flaky tests, report on failures, and find test suite problems.
   - :package: [JS Bundle Analysis](https://docs.codecov.com/docs/javascript-bundle-analysis): Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
   </details>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] fix(ci): prevent flaky keyring tests by deferring unlock to test phase [iggy]

Posted by "hubcio (via GitHub)" <gi...@apache.org>.
hubcio merged PR #2995:
URL: https://github.com/apache/iggy/pull/2995


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@iggy.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org