Skip to content

RUM-10224: Add octo sts policy for read access#2741

Merged
aleksandr-gringauz merged 1 commit into
developfrom
aleksandr-gringauz/RUM-10224/add-octo-sts-policy-for-read-access
Jun 19, 2025
Merged

RUM-10224: Add octo sts policy for read access#2741
aleksandr-gringauz merged 1 commit into
developfrom
aleksandr-gringauz/RUM-10224/add-octo-sts-policy-for-read-access

Conversation

@aleksandr-gringauz

@aleksandr-gringauz aleksandr-gringauz commented Jun 19, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

This policy is for GitLab CI in dd-sdk-android to be able to access release info about our own repo (for this).

Docs for dd-octo-sts.

I am following this example, but putting * so that any branch could access.

For creating PRs in shopist and mobile app I plan to create policies in their repos.

I will start actually using dd-octo-sts for check-release-pipeline in the next pr, because the policy yamls only from default branches are checked (link). And our default branch is develop.

I have no idea if this file works and there is no way to test.

I'll have to test after merging.

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@codecov-commenter

codecov-commenter commented Jun 19, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.75%. Comparing base (88008aa) to head (96e067d).
⚠️ Report is 1964 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2741      +/-   ##
===========================================
- Coverage    69.77%   69.75%   -0.01%     
===========================================
  Files          824      824              
  Lines        30790    30790              
  Branches      5177     5177              
===========================================
- Hits         21481    21477       -4     
- Misses        7844     7850       +6     
+ Partials      1465     1463       -2     

see 36 files with indirect coverage changes

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

@aleksandr-gringauz
aleksandr-gringauz force-pushed the aleksandr-gringauz/RUM-10224/add-octo-sts-policy-for-read-access branch 4 times, most recently from 8033404 to c143400 Compare June 19, 2025 11:25
Base automatically changed from aleksandr-gringauz/RUM-10224/add-dd-octo-sts-to-dockerfile to develop June 19, 2025 11:27
@aleksandr-gringauz
aleksandr-gringauz force-pushed the aleksandr-gringauz/RUM-10224/add-octo-sts-policy-for-read-access branch 6 times, most recently from 164f674 to 90fe462 Compare June 19, 2025 11:43
@aleksandr-gringauz
aleksandr-gringauz force-pushed the aleksandr-gringauz/RUM-10224/add-octo-sts-policy-for-read-access branch from 90fe462 to 96e067d Compare June 19, 2025 11:44
@aleksandr-gringauz
aleksandr-gringauz marked this pull request as ready for review June 19, 2025 11:45
@aleksandr-gringauz
aleksandr-gringauz requested review from a team as code owners June 19, 2025 11:45
@aleksandr-gringauz
aleksandr-gringauz merged commit 36cefc8 into develop Jun 19, 2025
14 of 16 checks passed
@aleksandr-gringauz
aleksandr-gringauz deleted the aleksandr-gringauz/RUM-10224/add-octo-sts-policy-for-read-access branch June 19, 2025 12:09
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.

4 participants