Nikolay Kolev

Results 137 comments of Nikolay Kolev
trafficstars

Obviously, subprocess32 is Python 2.x-only.

@jbbarth I started a PR, but I faced locally similar issues. Well, obviously I can't use simpleflow for now as I'm tied to Python 3. I'm not sure why people...

Also, when in `/my-account/wallet`, the text should be just *Topup*, *Transfer*, and *Transactions* - no need to have the *Wallet* prefix as, duh, everything on this screen is wallet-specific -...

Also, don't use `!!` - the standard is either one `!` or `!!!` - no such thing as `!!` in English!

I personally think that inline provider declaration, which honors the module `for_each` or `count` is the cleanest solution: ``` module "some_module" { source = "./some-module" for_each = local.modules_elements provider "provider1"...

@vivanov-dp This was pseudocode just to illustrate my point, which was that the logic of how the provider should be initialized could be encapsulated in the module. I can't think...

@vivanov-dp The ideal approach is to have identical code and only data, which varies between environment and clusters within the environment. Right now, almost everything has `for_each`/`count` except providers.

@jon-walton Your example is identical to mine, but I illustrated if the module needs more than a single provider.

@jon-walton Fair enough, we need dynamic providers - one way or another. Right now providers and outputs are the only two static resources in Terraform.

@apparentlymart Having `providerconfig(aws)` is a bit limiting as you can't pass the dynamic index from a TFC variable or `terraform.tfvars.json` file. The easiest and probably quickest to implement it just...