Skip to content

tests: Temporarily skip timerange verification tests#13244

Merged
jdamore-linode merged 2 commits intolinode:developfrom
jdamore-linode:skip-timerange-verification-tests
Jan 6, 2026
Merged

tests: Temporarily skip timerange verification tests#13244
jdamore-linode merged 2 commits intolinode:developfrom
jdamore-linode:skip-timerange-verification-tests

Conversation

@jdamore-linode
Copy link
Contributor

Description 📝

This temporarily skips the tests in timerange-verification.spec.ts. These tests fail regularly throughout the day and on multiple days throughout the year, and lately the failures and subsequent delay (45+ min) have been interfering with unrelated test runs. I'm going to temporarily skip these tests in hopes of avoiding complications with IAM testing and our upcoming release.

(cc @agorthi-akamai, unable to add you as a reviewer)

Changes 🔄

  • Skip the tests in timerange-verification.spec.ts
  • Add explanation comment describing why the tests are skipped and listing some potential next steps

Scope 🚢

Upon production release, changes in this PR will be visible to:

  • All customers
  • Some customers (e.g. in Beta or Limited Availability)
  • No customers / Not applicable

Target release date 🗓️

Monday, Jan 12, 2026

How to test 🧪

CI can cover this.

Author Checklists

As an Author, to speed up the review process, I considered 🤔

👀 Doing a self review
❔ Our contribution guidelines
🤏 Splitting feature into small PRs
➕ Adding a changeset
🧪 Providing/improving test coverage
🔐 Removing all sensitive information from the code and PR description
🚩 Using a feature flag to protect the release
👣 Providing comprehensive reproduction steps
📑 Providing or updating our documentation
🕛 Scheduling a pair reviewing session
📱 Providing mobile support
♿ Providing accessibility support


  • I have read and considered all applicable items listed above.

As an Author, before moving this PR from Draft to Open, I confirmed ✅

  • All tests and CI checks are passing
  • TypeScript compilation succeeded without errors
  • Code passes all linting rules

@jdamore-linode jdamore-linode requested a review from a team as a code owner January 6, 2026 16:33
@jdamore-linode jdamore-linode added Add'tl Approval Needed Waiting on another approval! and removed Missing Changeset labels Jan 6, 2026
@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🎉 860 passing tests on test run #2 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
0 Failing860 Passing11 Skipped41m 6s

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Jan 6, 2026
@jdamore-linode jdamore-linode merged commit 1991b9c into linode:develop Jan 6, 2026
35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Jan 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Add'tl Approval Needed Waiting on another approval! Ready for Review

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants