You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@texera.apache.org by "Yicong-Huang (via GitHub)" <gi...@apache.org> on 2026/06/06 21:36:50 UTC

[PR] chore(asf): use raw payload bypass_actors for branch protection [texera]

Yicong-Huang opened a new pull request, #5413:
URL: https://github.com/apache/texera/pull/5413

   ### What changes were proposed in this PR?
   
   Convert the `Default Branch Protection` ruleset in `.asf.yaml` from convenience syntax to raw payload so individual users can actually be registered as bypass actors — the convenience parser resolves `bypass_teams` via `get_team_by_slug(...)`, leaving a username unresolved (live `bypass_actors` on the ruleset is currently `null`). Raw payload also skips the parser's `_to_ref()` prefix step, so branch include patterns are spelled out in full (`refs/heads/release/*`, `refs/heads/rel/*`).
   
   ### Any related issues, documentation, discussions?
   
   Closes #5412. Follow-up to #5354 and INFRA-27977 (https://issues.apache.org/jira/browse/INFRA-27977).
   
   ### How was this PR tested?
   
   Normalized `.asf.yaml` through `apache/infrastructure-asfyaml`'s real `normalize_rulesets` and `configure_rulesets` (FakeRequester harness from its own test suite); the resulting POST payload matches the live ruleset shape on `apache/texera`.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Opus 4.7


-- 
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: notifications-unsubscribe@texera.apache.org

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


Re: [PR] chore(asf): use raw payload bypass_actors for branch protection [texera]

Posted by "codecov-commenter (via GitHub)" <gi...@apache.org>.
codecov-commenter commented on PR #5413:
URL: https://github.com/apache/texera/pull/5413#issuecomment-4640429216

   ## [Codecov](https://app.codecov.io/gh/apache/texera/pull/5413?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 51.64%. Comparing base ([`e9f2b8e`](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)) to head ([`b8e479b`](https://app.codecov.io/gh/apache/texera/commit/b8e479bd92eb46d700dd180b0b8895a5aff53ca0?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache)).
   
   <details><summary>Additional details and impacted files</summary>
   
   
   
   ```diff
   @@            Coverage Diff            @@
   ##               main    #5413   +/-   ##
   =========================================
     Coverage     51.64%   51.64%           
     Complexity     2420     2420           
   =========================================
     Files          1058     1058           
     Lines         41073    41073           
     Branches       4415     4415           
   =========================================
     Hits          21212    21212           
     Misses        18618    18618           
     Partials       1243     1243           
   ```
   
   | [Flag](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Coverage Δ | | *Carryforward flag |
   |---|---|---|---|
   | [access-control-service](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `41.89% <ø> (ø)` | | Carriedforward from [e9f2b8e](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [agent-service](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `33.76% <ø> (ø)` | | |
   | [amber](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `52.22% <ø> (ø)` | | Carriedforward from [e9f2b8e](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [computing-unit-managing-service](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `1.38% <ø> (ø)` | | Carriedforward from [e9f2b8e](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [config-service](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `55.38% <ø> (ø)` | | Carriedforward from [e9f2b8e](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [file-service](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `38.42% <ø> (ø)` | | Carriedforward from [e9f2b8e](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [frontend](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `46.40% <ø> (ø)` | | Carriedforward from [e9f2b8e](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [pyamber](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `90.69% <ø> (ø)` | | Carriedforward from [e9f2b8e](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [python](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `90.84% <ø> (ø)` | | Carriedforward from [e9f2b8e](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [workflow-compiling-service](https://app.codecov.io/gh/apache/texera/pull/5413/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | `58.39% <ø> (ø)` | | Carriedforward from [e9f2b8e](https://app.codecov.io/gh/apache/texera/commit/e9f2b8ed35679d2fa0338f7bbf74865b9a1f4246?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   
   *This pull request uses carry forward flags. [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) to find out more.
   </details>
   
   [:umbrella: View full report in Codecov by Harness](https://app.codecov.io/gh/apache/texera/pull/5413?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).   
   :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   <details><summary> :rocket: New features to boost your workflow: </summary>
   
   - :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: notifications-unsubscribe@texera.apache.org

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


Re: [PR] chore(asf): use raw payload bypass_actors for branch protection [texera]

Posted by "Yicong-Huang (via GitHub)" <gi...@apache.org>.
Yicong-Huang merged PR #5413:
URL: https://github.com/apache/texera/pull/5413


-- 
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: notifications-unsubscribe@texera.apache.org

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