Skip to content

Update infrastructures manifest#868

Open
n-boshnakov wants to merge 1 commit intogardener:masterfrom
n-boshnakov:update-manifest-2026.02
Open

Update infrastructures manifest#868
n-boshnakov wants to merge 1 commit intogardener:masterfrom
n-boshnakov:update-manifest-2026.02

Conversation

@n-boshnakov
Copy link
Contributor

@n-boshnakov n-boshnakov commented Feb 25, 2026

How to categorize this PR?

/kind enhancement

What this PR does / why we need it:
This PR updates the infrastructure.yaml file in order to remove unnecessary content.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:
To be merged after:

Summary by CodeRabbit

  • Documentation
    • Streamlined provider documentation by removing outdated tutorial sections from multiple infrastructure providers.
    • Reorganized documentation structure to improve clarity and maintainability.

@n-boshnakov n-boshnakov requested a review from a team as a code owner February 25, 2026 11:51
@gardener-prow gardener-prow bot added kind/enhancement Enhancement, improvement, extension cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. labels Feb 25, 2026
@n-boshnakov n-boshnakov changed the title Updated infrastructures yaml Update infrastructures manifest Feb 25, 2026
@gardener-prow gardener-prow bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Feb 25, 2026
@gardener-prow
Copy link

gardener-prow bot commented Feb 25, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign n-boshnakov for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@federated-github-access federated-github-access bot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Feb 25, 2026
@github-actions github-actions bot added needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. and removed ok-to-test Indicates a non-member PR verified by an org member that is safe to test. labels Feb 25, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2026

🚀 Preview Deployed

Your preview is being built and will be available at:
https://documentation-demo.gardener.cloud/pr-preview/pr-868/

Note: The preview will only be available after the GitHub Pages deployment completes in the demo repository.

To view the deployment status, visit: https://github.com/gardener/documentation-demo/actions

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 25, 2026

📝 Walkthrough

Walkthrough

Documentation configuration file updated to remove tutorials subsections from four cloud provider infrastructure modules (AWS, GCP, Azure, Alicloud) and introduce a proposals file reference in the GCP provider configuration.

Changes

Cohort / File(s) Summary
Documentation Configuration
.docforge/documentation/gardener-extensions/infrastructure.yaml
Removed tutorials subsections from gardener-extension-provider-aws, gardener-extension-provider-gcp, gardener-extension-provider-azure, and gardener-extension-provider-alicloud. Added proposals/datadisk-image-restore.md file reference in GCP provider with inline note regarding minimal value contribution.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 The tutorials hop away, they've had their day,
While proposals take their place in the grand display.
Four providers bid farewell to lessons past,
As documentation gardens shift at last!
✨📚

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Update infrastructures manifest' is vague and generic. While it refers to the infrastructures.yaml file being modified, it does not clearly specify what the actual change is (removing tutorials subsections from provider configurations). Consider using a more specific title such as 'Remove tutorials subsections from provider infrastructure configs' or 'Clean up infrastructure manifest by removing tutorial entries' to better communicate the primary change.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The PR description follows the required template structure with categorization, purpose explanation, and special notes included.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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

@federated-github-access federated-github-access bot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Feb 25, 2026
@github-actions github-actions bot removed the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. kind/enhancement Enhancement, improvement, extension needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant