Skip to content
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

Dotnet Example, fixes #306 #308

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Dotnet Example, fixes #306 #308

wants to merge 1 commit into from

Conversation

rshade
Copy link
Contributor

@rshade rshade commented Jan 17, 2025

This is a PR for the fix for #306, just contains the repro at the moment. I will add the issue stuff in a subsequent PR.

Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

@rshade rshade force-pushed the dotnet_example branch 6 times, most recently from 9f6caf7 to ef9cf03 Compare January 22, 2025 19:08
@rshade
Copy link
Contributor Author

rshade commented Jan 22, 2025

pulumi up --yes
Previewing update (dev)


     Type                                          Name                            Plan       Info
 +   pulumi:pulumi:Stack                           simple-cert-manager-dotnet-dev  create     1 warning
 +   ├─ random:index:RandomString                  random                          create
 +   ├─ kubernetes:core/v1:Namespace               sandbox-ns                      create
 +   └─ kubernetes-cert-manager:index:CertManager  cert-manager                    create
 +      └─ kubernetes:helm.sh/v3:Release           cert-manager-helm               create

Diagnostics:
  pulumi:pulumi:Stack (simple-cert-manager-dotnet-dev):
    warning: using pulumi-resource-kubernetes-cert-manager from $PATH at bin/pulumi-resource-kubernetes-cert-manager

Outputs:
    certManagerName     : "urn:pulumi:dev::simple-cert-manager-dotnet::kubernetes-cert-manager:index:CertManager::cert-manager"
    certManagerNamespace: "sandbox-ns-96d6f3f6"

Resources:
    + 5 to create

Updating (dev)


     Type                                          Name                            Status              Info
 +   pulumi:pulumi:Stack                           simple-cert-manager-dotnet-dev  created (44s)       1 warning
 +   ├─ random:index:RandomString                  random                          created (0.53s)
 +   ├─ kubernetes:core/v1:Namespace               sandbox-ns                      created (0.62s)
 +   └─ kubernetes-cert-manager:index:CertManager  cert-manager                    created (39s)
 +      └─ kubernetes:helm.sh/v3:Release           cert-manager-helm               created (36s)

Diagnostics:
  pulumi:pulumi:Stack (simple-cert-manager-dotnet-dev):
    warning: using pulumi-resource-kubernetes-cert-manager from $PATH at bin/pulumi-resource-kubernetes-cert-manager


Outputs:
    certManagerName     : "urn:pulumi:dev::simple-cert-manager-dotnet::kubernetes-cert-manager:index:CertManager::cert-manager"
    certManagerNamespace: "sandbox-ns-ed2ef472"

Resources:
    + 5 created

Duration: 47s

@rshade rshade force-pushed the dotnet_example branch 17 times, most recently from b561257 to 3a3f6f8 Compare January 23, 2025 15:03
@rshade
Copy link
Contributor Author

rshade commented Jan 23, 2025

@EronWright @Zaid-Ajaj I think its ready for review. After some discussion with @iwahbe, we think it will take a major change to pulumi-go-helmbase to get helmOptions to be a non-plain type.

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.

1 participant