Skip to content

Conversation

@josephcardillo-akamai
Copy link
Contributor

@josephcardillo-akamai josephcardillo-akamai commented Aug 5, 2025

Description 📝

Adds three new graph database marketplace applications to the Cloud Manager:

  • ArangoDB (ID: 1873119) - Multi-model database combining graph, document, and key-value capabilities
  • Memgraph (ID: 1878106) - High-performance in-memory graph database for real-time applications
  • Neo4j (ID: 1884304) - Leading graph database for connected data applications

Changes 🔄

  • Added app entries to oneClickApps.ts with proper descriptions, categories, colors, and related guides
  • Added both colored and white SVG assets for all three applications
  • All entries marked as `isNew: true` to display the "New" badge

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

Next available release

Preview 📷

I was unfortunately not able to run the dev environment for Cloud Manager following the instructions in https://linode.github.io/manager/GETTING_STARTED.html

It loads without issue or errors, but when I visit localhost:3000 it's just a blank screen.

[manager]   VITE v6.3.4  ready in 835 ms
[manager]
[manager]   ➜  Local:   http://localhost:3000/
[manager]   ➜  Network: use --host to expose
[api-v4]
[api-v4] 5:02:54 PM - Found 0 errors. Watching for file changes.
[validation]
[validation] 5:02:55 PM - Found 0 errors. Watching for file changes.
[ui]
[ui] 5:02:56 PM - Found 0 errors. Watching for file changes.
[shared]
[shared] 5:02:56 PM - Found 0 errors. Watching for file changes.
[manager] src/components/TransferDisplay/utils.test.tsx(25,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/Alerts/AlertRegions/AlertRegions.test.tsx(26,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/Alerts/AlertRegions/AlertRegions.test.tsx(31,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/Alerts/AlertRegions/AlertRegions.test.tsx(36,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/Alerts/AlertsDetail/AlertDetail.test.tsx(51,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/Alerts/AlertsResources/AlertsResources.test.tsx(19,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/Alerts/CreateAlert/CreateAlertDefinition.test.tsx(15,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/Alerts/CreateAlert/GeneralInformation/AlertEntityScopeSelect.test.tsx(15,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/Alerts/CreateAlert/Resources/CloudPulseModifyAlertResources.test.tsx(47,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/Alerts/EditAlert/EditAlertResources.test.tsx(57,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/CloudPulse/shared/CloudPulseRegionSelect.test.tsx(86,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/ObjectStorage/AccessKeyLanding/HostNamesDrawer.test.tsx(26,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/ObjectStorage/BucketLanding/BucketRegions.test.tsx(30,3): error TS2698: Spread types may only be created from object types.
[manager] src/features/PlacementGroups/PlacementGroupsDetailPanel.test.tsx(20,3): error TS2698: Spread types may only be created from object types.
[manager]
[manager] 5:03:03 PM - Found 14 errors. Watching for file changes.

🔒 Use the Mask Sensitive Data setting for security.

💡 For changes requiring multiple steps to validate, prefer a video for clarity.

Before After
📷 📷

How to test 🧪

Verification steps

(How to verify changes)

  • Confirm added app entries to oneClickApps.ts with proper descriptions, categories, colors, and related guides
  • Confirm both colored and white SVG assets for all three applications
  • Confirm all entries marked as `isNew: true` to display the "New" badge / show up in the New Apps section
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

@linode-gh-bot
Copy link
Collaborator

Cloud Manager UI test results

🔺 2 failing tests on test run #1 ↗︎

❌ Failing✅ Passing↪️ Skipped🕐 Duration
2 Failing703 Passing4 Skipped123m 48s

Details

Failing Tests
SpecTest
linode-storage.spec.tsCloud Manager Cypress Tests→linode storage tab » delete disk
qemu-reboot-upgrade-notice.spec.tsCloud Manager Cypress Tests→QEMU reboot upgrade notification » should display maintenance banner in 'Linode' landing page when one or more Linodes get impacted.

Troubleshooting

Use this command to re-run the failing tests:

pnpm cy:run -s "cypress/e2e/core/linodes/linode-storage.spec.ts,cypress/e2e/core/notificationsAndEvents/qemu-reboot-upgrade-notice.spec.ts"

Copy link
Contributor

@mjac0bs mjac0bs left a comment

Choose a reason for hiding this comment

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

✅ Confirmed:

  • New apps display in Marketplace
  • Icons are correct in light/dark mode
  • Links are redirect correctly or will redirect correctly once the guides are published (in the case of Memgraph and Neo4j)

Screenshot 2025-08-06 at 8 10 54 AM
Screenshot 2025-08-06 at 8 11 05 AM

@josephcardillo-akamai - It looks like you encountered console errors when trying to run locally. They might have been resolved by a pnpm clean followed by a pnpm bootstrap && pnpm dev. If you're ever having issues with getting Cloud running locally and want a quick visual confirmation, once the PR is created and CI has run, we generate preview links - reach out if you don't know how to access those.

@mjac0bs mjac0bs added Add'tl Approval Needed Waiting on another approval! Missing Changeset labels Aug 6, 2025
Copy link
Contributor

@coliu-akamai coliu-akamai left a comment

Choose a reason for hiding this comment

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

✅ confirmed apps exist, icons look good, and redirects

thanks @josephcardillo-akamai! + thanks @mjac0bs for the changeset!

@github-project-automation github-project-automation bot moved this from Review to Approved in Cloud Manager Aug 6, 2025
@coliu-akamai coliu-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Aug 6, 2025
@mjac0bs
Copy link
Contributor

mjac0bs commented Aug 6, 2025

@josephcardillo-akamai We will be able to get this in to the 8/12 Cloud Manager release next Tuesday.

Merging now - since this PR passed CI the first time except for known, unrelated Cy failures and the only thing I added in the last commit was a changeset, we don't need to wait another 40 minutes for Cypress tests to pass again.

@mjac0bs mjac0bs merged commit d6bc28b into linode:develop Aug 6, 2025
34 of 35 checks passed
@github-project-automation github-project-automation bot moved this from Approved to Merged in Cloud Manager Aug 6, 2025
@josephcardillo-akamai
Copy link
Contributor Author

Thank you @mjac0bs - I will reach out next time before hand if I'm having any issues with testing.

abailly-akamai pushed a commit to abailly-akamai/manager that referenced this pull request Aug 7, 2025
* feat: [OCA-1567] - Adds August 2025 Marketplace apps

* Added changeset: ArangoDB, Memgraph, and Neo4j apps to Marketplace

---------

Co-authored-by: mjac0bs <mjacobs@akamai.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! Marketplace

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants