Merge pull request #11220 from linode/release-v1.132.0

Release v1.132.0 - `release` → `staging`

.github/workflows/coverage_comment.yml

@@ -11,6 +11,7 @@ permissions:
 
 jobs:
   comment:
+    if: ${{ github.event.workflow_run.conclusion == 'success' }}
     runs-on: ubuntu-latest
 
     steps:

docker-compose.yml

@@ -117,10 +117,14 @@ services:
       <<: *default-env
       MANAGER_OAUTH: ${MANAGER_OAUTH}
 
-  # Cypress test runner service to run tests against a locally-served Cloud instance.
+  # Cypress test runner service to run tests against a local Cloud instance.
   #
-  # This is useful when testing against a customized or in-development build of
-  # Cloud Manager.
+  # This is useful when testing against a Cloud Manager instance served locally at
+  # `localhost:3000`, e.g. during development.
+  #
+  # If the local Cloud Manager instance is not served at `localhost:3000` (when
+  # served from a container, for example), prefer the `cypress_containerized`
+  # service instead.
   cypress_local:
     <<: *default-runner
     environment:
@@ -130,6 +134,38 @@ services:
       web:
         condition: service_healthy
 
+  # Cypress test runner service to run tests against a containerized Cloud instance.
+  #
+  # This service reverse proxies the given $CYPRESS_BASE_URL to `localhost:3000`.
+  # This is necessary for certain tests which require a secure context, which
+  # can typically only be achieved when Cloud is served at `localhost` or
+  # remotely behind SSL.
+  #
+  # For more information, refer to:
+  # https://developer.mozilla.org/en-US/docs/Web/Security/Secure_Contexts
+  #
+  # If the local Cloud Manager instance is served at `localhost:3000` (when
+  # running Cloud Manager locally during development, for example), prefer the
+  # `cypress_local` service instead.
+  cypress_containerized:
+    <<: *default-runner
+    build:
+      context: .
+      dockerfile: ./packages/manager/Dockerfile
+      target: e2e-reverse-proxy
+    environment:
+      <<: *default-env
+      MANAGER_OAUTH: ${MANAGER_OAUTH}
+      CYPRESS_BASE_URL: "http://localhost:3000"
+      REVERSE_PROXY_URL: ${CYPRESS_BASE_URL}
+    depends_on:
+      web:
+        condition: service_healthy
+    entrypoint:
+      - "/bin/sh"
+      - "-c"
+      - "caddy reverse-proxy --from $${CYPRESS_BASE_URL} --to $${REVERSE_PROXY_URL} & yarn $0 $@"
+
   # Cypress component test runner service.
   #
   # Unlike other Cloud Manager Cypress tests, these tests can be run without

docs/CONTRIBUTING.md

@@ -27,7 +27,8 @@ Feel free to open an issue to report a bug or request a feature.
     **Example:** `feat: [M3-1234] - Allow user to view their login history`
 
 6. Open a pull request against `develop` and make sure the title follows the same format as the commit message.
-7. If needed, create a changeset to populate our changelog.
+7. Keep in mind that our repository is public and open source! Before adding screenshots to your PR, we recommend you enable the **Mask Sensitive Data** setting in Cloud Manager [Profile Settings](https://cloud.linode.com/profile/settings).
+8. If needed, create a changeset to populate our changelog.
     - If you don't have the Github CLI installed or need to update it (you need GH CLI 2.21.0 or greater),
         - install it via `brew`: https://github.com/cli/cli#installation or upgrade with `brew upgrade gh`
         - Once installed, run `gh repo set-default` and pick `linode/manager` (only > 2.21.0)

docs/PULL_REQUEST_TEMPLATE.md

@@ -10,7 +10,9 @@ List any change relevant to the reviewer.
 Please specify a release date to guarantee timely review of this PR. If exact date is not known, please approximate and update it as needed.
 
 ## Preview 📷
-**Include a screenshot or screen recording of the change**
+**Include a screenshot or screen recording of the change.**
+
+:lock: Use the [Mask Sensitive Data](https://cloud.linode.com/profile/settings) setting for security.
 
 :bulb: Use `<video src="" />` tag when including recordings in table.
 

docs/development-guide/02-component-structure.md

@@ -15,9 +15,9 @@ The basic structure of a component file should follow:
 Here is a minimal code example demonstrating the basic structure of a component file:
 
 ```tsx
-import * as React from "react";
+import { omittedProps } from "@linode/ui";
 import { styled } from "@mui/material/styles";
-import { omittedProps } from "src/utilities/omittedProps";
+import * as React from "react";
 
 // If not exported, it can just be named `Props`
 export interface SayHelloProps {
@@ -66,6 +66,10 @@ When building a large component, it is recommended to break it down and avoid wr
 Components should, in most cases, come with their own unit test, although they can be skipped if an e2e suite is covering the functionality.
 Utilities should almost always feature a unit test.
 
+#### Security
+
+Consider whether the component is displaying data that may be sensitive, such as IP addresses or personal contact information. If so, make use of the `MaskableText` component or `masked` property of the `CopyTooltip` to hide this data for users who choose to 'Mask Sensitive Data' via Profile Settings.
+
 #### Styles
 
 - With the transition to MUI v5, the [`styled`](https://mui.com/system/styled/) API, along with the [`sx` prop](https://mui.com/system/getting-started/the-sx-prop/), is the preferred way to specify component-specific styles.

docs/development-guide/08-testing.md

@@ -19,7 +19,7 @@ yarn test
 Or you can run the tests in watch mode with:
 
 ```
-yarn test --watch
+yarn test:watch
 ```
 
 To run a specific file or files in a directory:

package.json

@@ -4,7 +4,8 @@
   "license": "Apache-2.0",
   "devDependencies": {
     "husky": "^9.1.6",
-    "typescript": "^5.5.4"
+    "typescript": "^5.5.4",
+    "vitest": "^2.1.1"
   },
   "scripts": {
     "lint": "yarn run eslint . --quiet --ext .js,.ts,.tsx",
@@ -23,7 +24,12 @@
     "start:manager": "yarn workspace linode-manager start",
     "start:manager:ci": "yarn workspace linode-manager start:ci",
     "clean": "rm -rf node_modules && rm -rf packages/@linode/api-v4/node_modules && rm -rf packages/manager/node_modules && rm -rf packages/@linode/validation/node_modules",
-    "test": "yarn workspace linode-manager test",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "test:manager": "yarn workspace linode-manager test",
+    "test:sdk": "yarn workspace @linode/api-v4 test",
+    "test:search": "yarn workspace @linode/search test",
+    "test:ui": "yarn workspace @linode/ui test",
     "package-versions": "node ./scripts/package-versions/index.js",
     "storybook": "yarn workspace linode-manager storybook",
     "cy:run": "yarn workspace linode-manager cy:run",

packages/api-v4/CHANGELOG.md

@@ -1,3 +1,22 @@
+## [2024-11-12] - v0.130.0
+
+
+### Added:
+
+- DBaaS: Suspend and Resume backend calls ([#11152](https://github.com/linode/manager/pull/11152))
+
+### Removed:
+
+- DBaaS: Deprecated types including MongoDB and Redis ([#11218](https://github.com/linode/manager/pull/11218))
+
+### Tech Stories:
+
+- Remove `@types/node` dependency ([#11157](https://github.com/linode/manager/pull/11157))
+
+### Upcoming Features:
+
+- DBaaS: Modify update payload to include version, add patch API ([#11196](https://github.com/linode/manager/pull/11196))
+
 
 ## [2024-10-28] - v0.129.0
 

packages/api-v4/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linode/api-v4",
-  "version": "0.129.0",
+  "version": "0.130.0",
   "homepage": "https://github.com/linode/manager/tree/develop/packages/api-v4",
   "bugs": {
     "url": "https://github.com/linode/manager/issues"
@@ -57,16 +57,14 @@
     "lib"
   ],
   "devDependencies": {
-    "@types/node": "^12.7.1",
     "@types/yup": "^0.29.13",
     "axios-mock-adapter": "^1.22.0",
     "concurrently": "^9.0.1",
     "eslint": "^6.8.0",
     "eslint-plugin-sonarjs": "^0.5.0",
     "lint-staged": "^15.2.9",
     "prettier": "~2.2.1",
-    "tsup": "^8.2.4",
-    "vitest": "^2.1.1"
+    "tsup": "^8.2.4"
   },
   "lint-staged": {
     "*.{ts,tsx,js}": [

packages/api-v4/src/databases/databases.ts

@@ -163,6 +163,21 @@ export const updateDatabase = (
     setData(data, updateDatabaseSchema)
   );
 
+/**
+ * patchDatabase
+ *
+ * Patch security updates for the database (outside of the maintenance window)
+ */
+export const patchDatabase = (engine: Engine, databaseID: number) =>
+  Request<void>(
+    setURL(
+      `${API_ROOT}/databases/${encodeURIComponent(
+        engine
+      )}/instances/${encodeURIComponent(databaseID)}/patch`
+    ),
+    setMethod('POST')
+  );
+
 /**
  * deleteDatabase
  *
@@ -301,3 +316,33 @@ export const getSSLFields = (engine: Engine, databaseID: number) =>
     ),
     setMethod('GET')
   );
+
+/**
+ * suspendDatabase
+ *
+ * Suspend the specified database cluster
+ */
+export const suspendDatabase = (engine: Engine, databaseID: number) =>
+  Request<{}>(
+    setURL(
+      `${API_ROOT}/databases/${encodeURIComponent(
+        engine
+      )}/instances/${encodeURIComponent(databaseID)}/suspend`
+    ),
+    setMethod('POST')
+  );
+
+/**
+ * resumeDatabase
+ *
+ * Resume the specified database cluster
+ */
+export const resumeDatabase = (engine: Engine, databaseID: number) =>
+  Request<{}>(
+    setURL(
+      `${API_ROOT}/databases/${encodeURIComponent(
+        engine
+      )}/instances/${encodeURIComponent(databaseID)}/resume`
+    ),
+    setMethod('POST')
+  );