Skip to content

feat(oraclecloud): support regionless SDK setup#11740

Merged
HugoPBrito merged 3 commits into
ci/oci-regionless-platform-11565from
feat/oci-regionless-sdk-chain-11565
Jul 6, 2026
Merged

feat(oraclecloud): support regionless SDK setup#11740
HugoPBrito merged 3 commits into
ci/oci-regionless-platform-11565from
feat/oci-regionless-sdk-chain-11565

Conversation

@HugoPBrito

@HugoPBrito HugoPBrito commented Jun 30, 2026

Copy link
Copy Markdown
Member

Context

Oracle Cloud credentials in Prowler can be valid without using a user-selected scan region. The SDK needs a safe internal region only to bootstrap the OCI client and discover subscribed regions.

Description

This PR updates the Oracle Cloud provider initialization so API-key authentication can fall back to the internal default region when no explicit scan region filter is provided. It also adds SDK tests for regionless credentials and documents the SDK change in the changelog.

Changed files:

File Change
prowler/CHANGELOG.md Adds the SDK changelog entry.
prowler/providers/oraclecloud/oraclecloud_provider.py Adds the regionless fallback behavior.
tests/providers/oraclecloud/oraclecloud_provider_test.py Adds SDK coverage for regionless OCI setup.

Steps to review

  1. Review the SDK diff against the platform slice.
  2. Confirm the SDK changes match the SDK portion of feat: support regionless OCI provider setup #11565.
  3. Confirm the API slice builds on this PR.

Validation previously run for this slice:

uv run pytest tests/providers/oraclecloud/oraclecloud_provider_test.py

Checklist

Community Checklist
  • This feature/issue is listed in here or roadmap.prowler.com
  • Is it assigned to me, if not, request it via the issue/feature in here or Prowler Community Slack

SDK/CLI

  • Are there new checks included in this PR? No
    • If so, do we need to update permissions for the provider? Please review this carefully.

UI

  • All issue/task requirements work as expected on the UI
  • If this PR adds or updates npm dependencies, include package-health evidence (maintenance, popularity, known vulnerabilities, license, release age) and explain why existing/native alternatives are insufficient.
  • Screenshots/Video of the functionality flow (if applicable) - Mobile (X < 640px)
  • Screenshots/Video of the functionality flow (if applicable) - Table (640px > X < 1024px)
  • Screenshots/Video of the functionality flow (if applicable) - Desktop (X > 1024px)
  • Ensure new entries are added to CHANGELOG.md, if applicable.

API

  • All issue/task requirements work as expected on the API
  • Endpoint response output (if applicable)
  • EXPLAIN ANALYZE output for new/modified queries or indexes (if applicable)
  • Performance test results (if applicable)
  • Any other relevant evidence of the implementation (if applicable)
  • Verify if API specs need to be regenerated.
  • Check if version updates are required (e.g., specs, uv, etc.).
  • Ensure new entries are added to CHANGELOG.md, if applicable.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Chain Context

Field Value
Chain OCI regionless provider setup split from #11565
Tracker PR #11565
Position 2 of 4
Base ci/oci-regionless-platform-11565
Depends on #11739
Follow-up #11741
Review budget 357 / 400
Starts at Platform CI guard slice
Ends with SDK regionless OCI provider setup

Chain Overview

master
 └── #11739 Platform CI guard
      └── 📍 #11740 SDK regionless OCI setup
           └── #11741 API regionless OCI credentials
                └── #11742 UI regionless OCI credentials

Scope

Autonomy

  • CI is expected to pass for this PR branch
  • This PR has one deliverable scope
  • This PR can be rolled back without unrelated changes
  • Tests, docs, or manual verification cover this unit

@HugoPBrito HugoPBrito requested a review from a team as a code owner June 30, 2026 14:12
@coderabbitai

coderabbitai Bot commented Jun 30, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: bce5bf61-13ff-4eb4-a9b9-5db2927b6ebd

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/oci-regionless-sdk-chain-11565

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Comment thread prowler/CHANGELOG.md

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Move this into a new section for 5.33.0

@HugoPBrito HugoPBrito merged commit 9785dba into ci/oci-regionless-platform-11565 Jul 6, 2026
4 checks passed
@HugoPBrito HugoPBrito deleted the feat/oci-regionless-sdk-chain-11565 branch July 6, 2026 13:16
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.

2 participants