search

pulumi / examples

Infrastructure, containers, and serverless apps to AWS, Azure, GCP, and Kubernetes... all deployed with Pulumi
https://www.pulumi.com
Apache License 2.0
2.33k stars 874 forks source link

Update programs to kubernetes 4.0.3 #1472

Closed mnlumi closed 4 months ago

mnlumi commented 11 months ago

Resolves https://github.com/pulumi/examples/issues/1466

mnlumi commented 11 months ago

Opened Panic in kubernetes-go-guestbook example program pulumi/docs#9744

mnlumi commented 10 months ago

@cnunciato Could you take a look at kubernetes-go-helm-wordpress? helmv2 has been removed from kubernetes v4, but updating to helmv3 and the chart version of wordpress is failing with an error.

 pulumi:pulumi:Stack (kubernetes-go-helm-wordpress-dev):
    panic: interface conversion: interface {} is nil, not *v1.Service
    goroutine 89 [running]:
    main.main.func1.1({0x0?, 0x0?})
scottslowe commented 9 months ago

Bumping this to M96 due to Helm v3 issue described above

scottslowe commented 9 months ago

~I think I have working code for this. It works as expected on EKS (after I fixed a problem with the EKS template); now testing on GKE.~

Update: it's not working, the behavior persists and is a bug: https://github.com/pulumi/pulumi-kubernetes/issues/2638

@mnlumi @cnunciato How would you like to proceed? Wait for a fix for the bug and update all the examples in one swoop, or update the other examples and fix this one example later?

mnlumi commented 8 months ago

@scottslowe At this point, the PR is now several releases behind for Kubernetes. My suggestion is to exclude the broken example while opening an issue to handle it separately.

scottslowe commented 8 months ago

@mnlumi I'm good with that approach, assuming no one else has objections.

scottslowe commented 8 months ago

@mnlumi Per Eric's e-mail, we'll leave this issue open to track resolution of the broken example. Just exclude the broken example from the PR to update the provider for all the rest of the examples. Once we have all tests passing, then we can merge that PR to get everything else updated.

mnlumi commented 4 months ago

Closing as this as it is significantly out of date and a new pull request would be more efficient. @EronWright @mjeffryes - You may want to work with your team or @cnunciato to get these kubernetes examples updated to v4 instead of v3 and the latest pulumi versions as well.