Remove duplicated pages to improve search results by marc1404 · Pull Request #840 · gardener/documentation

marc1404 · 2026-02-06T15:09:59Z

How to categorize this PR?

/kind cleanup enhancement task

What this PR does / why we need it:

In the Docforge configuration, certain files are "hand-picked" to replicate them in relevant, accessible locations such as the Guides or Security and Compliance section. However, since the replicated file remains at its original location (usually specified via the fileTree directive in Docforge), the content is effectively duplicated.
This leads to duplicate search results for certain queries targeting these pages.

This PR does the following:

Explicitly excludes hand-picked files in their original location (where they're included through fileTree).
Removes 3 duplicates where their original location is the semantically better place for the page to appear in.

Which issue(s) this PR fixes:
Fixes #720

Special notes for your reviewer:

It might be easier to review the changes by browsing the individual commits.

/cc @n-boshnakov @BoHristova @klocke-io

ℹ️ @LibenHailu, thank you for figuring out the root cause issue!

netlify · 2026-02-06T15:10:05Z

❌ Deploy Preview for soon-to-be-gardenercloud failed.

Name	Link
🔨 Latest commit	`16f3106`
🔍 Latest deploy log	https://app.netlify.com/projects/soon-to-be-gardenercloud/deploys/6986044bfc942f0007ed1c47

marc1404 · 2026-02-06T15:11:46Z

.docforge/documentation/gardener-extensions/infrastructure.yaml

-      title: Provider Alicloud
+      title: Provider Alibaba Cloud


Unrelated scout rule change

marc1404 · 2026-02-06T15:11:50Z

.docforge/documentation/gardener-extensions/infrastructure.yaml

-      title: Provider Openstack
+      title: Provider OpenStack


Unrelated scout rule change

gardener-robot · 2026-02-09T03:05:52Z

@BoHristova, @klocke-io, @n-boshnakov You have pull request review open invite, please check

marc1404 · 2026-02-09T07:48:42Z

.docforge/documentation/documentation.yaml

-        - file: _index.md
-          frontmatter:
-            title: High Availability
-            weight: 30


Only contains 2 files that are excluded below. Therefore, we don't need the directory when the files are shown in the Guides section instead.

marc1404 · 2026-02-09T07:50:18Z

.docforge/documentation/documentation.yaml

-    - dir: code
-      structure:
-      - dir: security-guide
-        structure:
-          - file: _index.md
-            frontmatter:
-              title: Security Release Process
-            source: https://github.com/gardener/.github/blob/main/SECURITY.md


Already included here:

documentation/.docforge/documentation/documentation.yaml

Line 26 in b984562

source: https://github.com/gardener/.github/blob/main/SECURITY.md

We should not have the page twice.

marc1404 · 2026-02-09T07:51:06Z

.docforge/documentation/other-components.yaml

-  - file: api-reference.md
-    frontmatter:
-      title: API Reference
-      persona: Developers
-    source: https://github.com/gardener/etcd-druid-api/blob/main/docs/api-reference/druid.md


Already included through the fileTree option directly below

marc1404 · 2026-02-09T07:52:14Z

.docforge/documentation/security-and-compliance.yaml

- file: /website/documentation/getting-started/features/credential-rotation.md
-  frontmatter:
-    title: Credential Rotation
-    weight: 37


Already included through the fileTree option on getting-started:

documentation/.docforge/documentation/documentation.yaml

Line 7 in 4aa3630

- fileTree: /website/documentation/getting-started

BoHristova · 2026-02-09T08:05:54Z

Thank you :)

/lgtm

gardener-prow · 2026-02-09T08:06:01Z

LGTM label has been added.

Details

Git tree hash: 53bfa749fbf05c9c34a24b22c6e0d2afc75d5f9f

n-boshnakov

/lgtm

Tested locally, everything seems to work fine.

gardener-prow · 2026-02-09T10:30:43Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: n-boshnakov

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

The pull request process is described here

Details

Needs approval from an approver in each of these files:

~~OWNERS~~ [n-boshnakov]

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

marc1404 added 5 commits February 6, 2026 16:02

docs: Exclude hand-picked guides in their original location

b66f641

docs: Remove duplicated security page

b984562

docs: Remove duplicated API reference page in etcd-druid

adf77ed

docs: Remove duplicated credential rotation page

f1f0bdd

docs: Improve titles of provider Alibaba Cloud and OpenStack

16f3106

marc1404 requested a review from a team as a code owner February 6, 2026 15:10

gardener-prow bot requested review from BoHristova, klocke-io and n-boshnakov February 6, 2026 15:10

gardener-prow bot added the kind/cleanup Something that is not needed anymore and can be cleaned up label Feb 6, 2026

gardener-prow bot added kind/enhancement Enhancement, improvement, extension kind/task General task cla: yes Indicates the PR's author has signed the cla-assistant.io CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 6, 2026

gardener-github-actions bot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Feb 6, 2026

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 6, 2026

gardener-github-actions bot added the ok-to-test Indicates a non-member PR verified by an org member that is safe to test. label Feb 6, 2026

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 6, 2026

marc1404 commented Feb 6, 2026

View reviewed changes

marc1404 commented Feb 9, 2026

View reviewed changes

gardener-prow bot added the lgtm Indicates that a PR is ready to be merged. label Feb 9, 2026

n-boshnakov approved these changes Feb 9, 2026

View reviewed changes

gardener-prow bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 9, 2026

marc1404 merged commit 6351d73 into gardener:master Feb 9, 2026
21 of 27 checks passed

marc1404 deleted the docs/720-remove-duplicates branch February 9, 2026 10:43

marc1404 mentioned this pull request Feb 9, 2026

Duplicate search results #842

Closed

Comments

Conversation

marc1404 commented Feb 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

netlify bot commented Feb 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

❌ Deploy Preview for soon-to-be-gardenercloud failed.

Uh oh!

marc1404 Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

marc1404 Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

gardener-robot commented Feb 9, 2026

Uh oh!

marc1404 Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

marc1404 Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

marc1404 Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

marc1404 Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

BoHristova commented Feb 9, 2026

Uh oh!

gardener-prow bot commented Feb 9, 2026

Uh oh!

n-boshnakov left a comment

Choose a reason for hiding this comment

Uh oh!

gardener-prow bot commented Feb 9, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

marc1404 commented Feb 6, 2026 •

edited

Loading

netlify bot commented Feb 6, 2026 •

edited

Loading