Skip to content

Conversation

@micheleRP
Copy link
Contributor

@micheleRP micheleRP commented Feb 28, 2025

Description

This pull request updates the required vCPU specifications for the Azure subscription used with Redpanda.

Resolves https://redpandadata.atlassian.net/browse/DOC-1052
Review deadline:

Page previews

Check Azure quota

Checks

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

@netlify
Copy link

netlify bot commented Feb 28, 2025

Deploy Preview for rp-cloud ready!

Name Link
🔨 Latest commit e1e6b91
🔍 Latest deploy log https://app.netlify.com/sites/rp-cloud/deploys/67c728064171cd00080d86ca
😎 Deploy Preview https://deploy-preview-222--rp-cloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@micheleRP micheleRP requested review from ligfx and ncole February 28, 2025 19:25
@micheleRP micheleRP marked this pull request as ready for review February 28, 2025 19:26
@micheleRP micheleRP requested a review from a team as a code owner February 28, 2025 19:26
Copy link
Contributor

@ligfx ligfx left a comment

Choose a reason for hiding this comment

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

DDv5-series

Should be "Ddv5-series", second "D" is lowercase

3 variable nodes for Tier 1

What's a variable node?

@ncole
Copy link
Contributor

ncole commented Feb 28, 2025

Should also replace in the restrictions text: Standard_L8as_v3 => Standard_D2d_v5

@ncole
Copy link
Contributor

ncole commented Feb 28, 2025

What's a variable node?

There are times when we do maintenance and a temporary node (1x per pool) is created. In this case, exactly 3 nodes would not provide enough CPU for said temporary nodes.

@micheleRP
Copy link
Contributor Author

What's a variable node?

There are times when we do maintenance and a temporary node (1x per pool) is created. In this case, exactly 3 nodes would not provide enough CPU for said temporary nodes.

Should we replace variable nodes with: "+ 3 additional compute nodes for Tier 1 workloads"?

@ligfx
Copy link
Contributor

ligfx commented Mar 4, 2025

Instead of + 3 variables nodes maybe something like + extra capacity for 3 more nodes, which may be utilized temporarily during maintenance?

Other than that, looks good to me.

@ncole
Copy link
Contributor

ncole commented Mar 4, 2025

^ this verbiage is ok by me. Other than that, LGTM.

@micheleRP
Copy link
Contributor Author

Instead of + 3 variables nodes maybe something like + extra capacity for 3 more nodes, which may be utilized temporarily during maintenance?

Other than that, looks good to me.

@ligfx: Is this OK: (3 Redpanda broker nodes + extra capacity for 3 more nodes that could be utilized temporarily during tier 1 maintenance)

@ligfx
Copy link
Contributor

ligfx commented Mar 4, 2025

@micheleRP sounds good to me!

@micheleRP micheleRP merged commit a6cac8f into main Mar 4, 2025
5 checks passed
@micheleRP micheleRP deleted the DOC-1052-Create-a-BYOC-Cluster-on-Azure-stills-references-old-instance-types branch March 4, 2025 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants