Skip to content

upcoming: [M3-10674] - Add pendo ids for VM analytics - #12983

Merged
jaalah-akamai merged 5 commits into
linode:developfrom
jaalah-akamai:M3-10674
Oct 15, 2025
Merged

jaalah-akamai merged 5 commits into
linode:developfrom
jaalah-akamai:M3-10674

Conversation

@jaalah-akamai

@jaalah-akamai jaalah-akamai commented Oct 13, 2025

Copy link
Copy Markdown
Contributor

Description 📝

Implements Pendo tracking tags for VM Host Maintenance Policy feature to monitor user interactions and gather analytics.

Changes 🔄

  • Added data-pendo-id to several areas for VM Host Maintenance
  • Added ability to add custom pendo id's to <Link> component
  • "Learn more" links also have pendo id's (not shown in screenshots below)

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 🗓️

10/21

Preview 📷

File Screenshot
Autocomplete, options, and account save button with id's Screenshot 2025-10-13 at 11 47 41 AM Screenshot 2025-10-13 at 11 40 22 AM Screenshot 2025-10-13 at 11 49 02 AM
Linode Details - Save Button Screenshot 2025-10-13 at 11 51 05 AM
Banners: Linode, Platform, Regular Maintenance Screenshot 2025-10-13 at 11 30 01 AMScreenshot 2025-10-13 at 11 33 53 AMScreenshot 2025-10-13 at 11 35 33 AM

|

How to test 🧪

Reproduction steps

  • In your local environment on develop
  • Choose a linode you have on your account, copy it's linode id
  • Open our dev-tools, enable MSW, and under "Presets" enable "Maintenance"
  • Edit the "Maintenance" preset
  • In the JSON editor, copy this object and replace the: id, label, and url with your chosen linode
[
  {
    "entity": {
      "id": <ADD_LINODE_ID>,
      "label": "<ADD_LINODE_LABEL>",
      "type": "linode",
      "url": "/v4/linode/instances/<ADD_LINODE_ID>"
    },
    "maintenance_policy_set": "linode/migrate",
    "reason": "We must replace faulty RAM in your Linode's host.",
    "description": "emergency",
    "source": "user",
    "status": "scheduled",
    "type": "live_migration",
    "when": null,
    "not_before": null,
    "start_time": null,
    "complete_time": null
  }
]

Platform Notification

  • Follow similar steps for Notifications preset
  • Choose "Add from template" > "Platform Maintenance Scheduled"
  • Add in linode id's
  • Ensure dates are in future

Verification steps

  • For Banners, follow reproduction steps above and you should see banners in linodes landing page.
  • For other changes to Maintenance Policy Select, find the component on:
    • /linodes/create/os
    • /linodes/<linode_id>/settings
    • /account-settings
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

@jaalah-akamai jaalah-akamai self-assigned this Oct 13, 2025
@jaalah-akamai jaalah-akamai added For Next Release Apply this label if the PR needs to be in the next release. Host & VM Maintenance labels Oct 13, 2025
@jaalah-akamai
jaalah-akamai marked this pull request as ready for review October 13, 2025 16:13
@jaalah-akamai
jaalah-akamai requested a review from a team as a code owner October 13, 2025 16:13
@jaalah-akamai
jaalah-akamai marked this pull request as draft October 13, 2025 16:15
@jaalah-akamai
jaalah-akamai marked this pull request as ready for review October 13, 2025 16:23

@dwiley-akamai dwiley-akamai left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code review ✅
Verification steps ✅

The E2E tests that failed seem a bit flaky; I ran them a few times locally and the failures were inconsistent.

@linode-gh-bot

Copy link
Copy Markdown

Cloud Manager UI test results

🔺 1 failing test on test run #5 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
1 Failing840 Passing11 Skipped39m 6s

Details

Failing Tests
SpecTest
timerange-verification.spec.tsCloud Manager Cypress Tests→Integration tests for verifying Cloudpulse custom and preset configurations » should implement and validate custom date/time picker for a specific date and time range

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/cloudpulse/timerange-verification.spec.ts"

@github-project-automation github-project-automation Bot moved this from Review to Approved in Cloud Manager Oct 14, 2025
@bnussman-akamai bnussman-akamai added the Approved Multiple approvals and ready to merge! label Oct 14, 2025
@jaalah-akamai
jaalah-akamai merged commit 52a6eda into linode:develop Oct 15, 2025
34 of 35 checks passed
@github-project-automation github-project-automation Bot moved this from Approved to Merged in Cloud Manager Oct 15, 2025
corya-akamai pushed a commit to corya-akamai/manager that referenced this pull request Dec 8, 2025
* upcoming: [M3-10674] - Add pendo ids for VM analytics

* Add learn more links

* Added changeset: Add pendo ids for VM Host Maintenance analytics

---------

Co-authored-by: Jaalah Ramos <jaalah.ramos@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Multiple approvals and ready to merge! For Next Release Apply this label if the PR needs to be in the next release. Host & VM Maintenance

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants