Add timeouts to quickstart tests to avoid failures

[JakeSCahill]

Description

Resolves #1397

In CI, it sometimes takes a bit longer for the page to load after logging in. This PR increases the default timeout of 5000 to 10000 to give the page time to load.

Page previews

Checks

• New feature
• Content gap
• Support Follow-up
• Small fix (typos, links, copyedits, etc)

[netlify]

✅ Deploy Preview for redpanda-docs-preview ready!

Name	Link
🔨 Latest commit	9a6e545
🔍 Latest deploy log	https://app.netlify.com/projects/redpanda-docs-preview/deploys/68f8afb0a4753500083f2703
😎 Deploy Preview	https://deploy-preview-1398--redpanda-docs-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

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.

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

📝 Walkthrough

Walkthrough

The pull request addresses a failing Doc Detective test by adding 10-second timeout parameters to UI interaction steps in the quick-start.adoc specification file. Specifically, timeouts are added to three login-related steps and the broker overview "View" button interaction steps. These timeouts allow the test framework to wait up to 10 seconds for elements to become available before attempting interaction, replacing reliance on immediate element detection.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

The change is a straightforward parameter addition across a single file with no logic modifications, new features, or structural changes. All updates follow an identical pattern of adding the same timeout value to existing step definitions, making this a homogeneous, low-complexity change with minimal risk.

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The pull request title "Add timeouts to quickstart tests to avoid failures" is clear and specific, directly summarizing the main change in the diff. The raw summary confirms that the modifications add 10000ms timeout parameters to multiple UI interaction steps in the quick-start.adoc file to prevent test failures. The title accurately represents this primary change without being vague or misleading.
Linked Issues Check	✅ Passed	The PR directly addresses issue #1397 by implementing one of the suggested remediations: adding a more robust wait/assertion mechanism through increased timeout values. The changes add 10000ms timeouts to UI interaction steps in the quickstart test, which aligns with the issue's objective to fix the Doc Detective test failure caused by slow page loads in CI environments. The timeout increase from 5000ms to 10000ms provides the page adequate time to load before attempting to locate elements, which directly resolves the core coding requirement to make tests more resilient to timing variations.
Out of Scope Changes Check	✅ Passed	All changes in the pull request are directly scoped to the linked issue #1397. The modifications add timeout parameters exclusively to the quick-start.adoc test file to address the specific test failure related to page load timing. No unrelated changes such as refactoring, documentation updates outside the test context, or modifications to other files are present. The changes are narrowly focused on the remediation of the failing Doc Detective test through increased timeout values.
Description Check	✅ Passed	The pull request description includes the essential required sections from the template: a Description section that references the GitHub issue (#1397) and explains the fix (increasing timeout from 5000ms to 10000ms to accommodate slower page loads in CI), a Page previews section with placeholder guidance, and a Checks section with categorization options. While the review deadline field is not filled and the description references a GitHub issue rather than a Jira ticket, the core content clearly communicates the purpose and rationale for the changes. The description is sufficiently complete to understand the fix being implemented.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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