-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Guide "Deploy a Linode Kubernetes Engine Cluster Using Pulumi" #6555
Conversation
✅ Deploy Preview for nostalgic-ptolemy-b01ab8 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Remove incorrect inclusion of docs domain name in link Signed-off-by: Scott Lowe <[email protected]>
I fixed a Markdown linting error with 699fa12, let me know if any other changes are needed before merging. Thank you! |
Hi @ringods ! I tried the instructions in the guide and I get an error.
|
@Rajakavitha1 I will have to update the Pulumi Linode templates. The templates still point to major v3 of our Linode package: While we are already far into the major v4 setup: https://github.com/pulumi/pulumi-linode/releases This is unrelated to the correctness of the guide. |
I created this PR to fix this: pulumi/templates#661 Local testing of my fix gives a better outcome:
|
@Rajakavitha1 the fix for our Pulumi Linode templates has been merged & verified on our side. Can you retry? |
Thank you @ringods . I will try the instructions again. I truly appreciate your prompt response. Looking forward to taking this PR to merge. |
Hi @ringods !! I tried the instructions and I get an error in the preview that reads: error: an unhandled error occurred: Program exited with non-zero exit code: 1. Here's the output from the terminal. Please let me know what am I missing. |
@Rajakavitha1 can you provide me the output of My first guess is that your NodeJS version is too old. The requirements are listed on our website: https://www.pulumi.com/docs/languages-sdks/javascript/ |
Hi @ringods here's the output of plume. about:
|
@Rajakavitha1 you are on Node v13: The requirements for NodeJS are listed in the opening paragraph here: https://www.pulumi.com/docs/languages-sdks/javascript/ NodeJS v18 is what I would advice. |
Hi @ringods ! I installed the latest Node and it works. Thank you. |
LGTM! |
Hi @andystevensname. I tested validated and this PR is ready for merge. |
Fix relative path to pass Markdown linting tests Signed-off-by: Scott Lowe <[email protected]>
@Rajakavitha1 When should we expect this content to go live on the Linode site? (It doesn't appear to be visible currently.) |
Hi @scottslowe . It is live here: https://www.linode.com/docs/products/compute/kubernetes/guides/deploy-cluster-using-pulumi/ published on October 25, 2023. |
Here is a guide on how to set up an LKE cluster using Pulumi and Typescript. I kept the structure similar to the TF guide. I also updated some of the links in other articles pointing to resources on the Pulumi website.