https://pulumi.com logo
Join the conversationJoin Slack
Channels
announcements
automation-api
aws
azure
blog-posts
built-with-pulumi
cloudengineering
cloudengineering-support
content-share
contribex
contribute
docs
dotnet
finops
general
getting-started
gitlab
golang
google-cloud
hackathon-03-19-2020
hacktoberfest
install
java
jobs
kubernetes
learn-pulumi-events
linen
localstack
multi-language-hackathon
office-hours
oracle-cloud-infrastructure
plugin-framework
pulumi-cdk
pulumi-crosscode
pulumi-deployments
pulumi-kubernetes-operator
pulumi-service
pulumiverse
python
registry
status
testingtesting123
testingtesting321
typescript
welcome
workshops
yaml
Powered by Linen
contribute
  • r

    rough-oil-1458

    09/02/2018, 7:55 PM
    MakeTerraformConfig
  • r

    rough-oil-1458

    09/02/2018, 7:55 PM
    Ill take a look in there
  • r

    rough-oil-1458

    09/02/2018, 7:55 PM
    but yea, I believe it did the correct thing in this case
    👍 1
  • r

    rough-oil-1458

    09/02/2018, 7:56 PM
    but.... the vsphere server, should be required
  • r

    rough-oil-1458

    09/02/2018, 7:56 PM
    Let me see if thats a bug in TF
  • b

    big-piano-35669

    09/02/2018, 7:58 PM
    These properties definitely look like they'll suffer from issue #181.
  • b

    big-piano-35669

    09/02/2018, 7:58 PM
    E.g. https://github.com/terraform-providers/terraform-provider-vsphere/blob/3bccdebdd9826fb8d191f7e71a9eeda2db81e97e/vsphere/provider.go#L34
  • b

    big-piano-35669

    09/02/2018, 7:58 PM
    They all use
    schema.EnvDefaultFunc
    . We don't respect that currently, so they'll all require you to manually set via
    pulumi config set
    .
  • r

    rough-oil-1458

    09/02/2018, 7:59 PM
    yea
  • b

    big-piano-35669

    09/02/2018, 7:59 PM
    As mentioned, pretty sure this will be fixed this week in time for 0.16, so for now if you're okay manually setting them, it should "just work" soon.
    w
    • 2
    • 1
  • r

    rough-oil-1458

    09/02/2018, 7:59 PM
    haha, its working 🙂
    🎉 1
  • r

    rough-oil-1458

    09/02/2018, 7:59 PM
    Im a happy camper
  • b

    big-piano-35669

    09/02/2018, 7:59 PM
    Great to hear! Me too, so excited to see this coming to life 🙂
  • r

    rough-oil-1458

    09/02/2018, 7:59 PM
    Just learning more about this, so i can convert more TF
  • r

    rough-oil-1458

    09/02/2018, 8:00 PM
    I'd like to have parity at work for Pulumi
  • r

    rough-oil-1458

    09/02/2018, 8:00 PM
    I can't stand learning arbitrary crazy opinions
  • r

    rough-oil-1458

    09/02/2018, 8:01 PM
    https://blog.gruntwork.io/terraform-tips-tricks-loops-if-statements-and-gotchas-f739bbae55f9
  • r

    rough-oil-1458

    09/02/2018, 8:01 PM
    LOL, Idk if you ever read that before
  • b

    big-piano-35669

    09/02/2018, 8:07 PM
    I have -- and it seems a little bonkers. This sort of thing motivated us pretty strongly early on with Pulumi 🙂
  • r

    rough-oil-1458

    09/02/2018, 8:53 PM
    omg yes
  • r

    rough-oil-1458

    09/02/2018, 8:53 PM
    everything seems to be working!
  • r

    rough-oil-1458

    09/02/2018, 8:54 PM
    ❤️
  • b

    big-piano-35669

    09/02/2018, 8:54 PM
    🍾
  • b

    big-piano-35669

    09/02/2018, 8:54 PM
    Congrats! Awesome!
  • r

    rough-oil-1458

    09/02/2018, 8:55 PM
    So what are these packages 😛 ?
  • b

    big-piano-35669

    09/02/2018, 8:55 PM
    The NPM packages?
  • b

    big-piano-35669

    09/02/2018, 8:56 PM
    I think we'll want to work with you to get this hooked up to publish as
    @pulumi/vsphere
    , if that's your ideal outcome. Similar to what we did with OpenStack: https://github.com/pulumi/pulumi-openstack
  • r

    rough-oil-1458

    09/02/2018, 8:57 PM
    Yea, I still have to write tests, clean up the code
    👍 1
  • r

    rough-oil-1458

    09/02/2018, 8:57 PM
    but I cant believe i went form 0
  • r

    rough-oil-1458

    09/02/2018, 8:57 PM
    to using npm install assert-plus, and just writing a function to deploy everything
Powered by Linen
Title
r

rough-oil-1458

09/02/2018, 8:57 PM
to using npm install assert-plus, and just writing a function to deploy everything
View count: 1