Antonio Gurgel

Results 7 comments of Antonio Gurgel

Can this repo be archived with a note in the README pointing to the new fork?

``` body = confluence.get_content_template(TEMPLATE_ID)["body"]["storage"]["value"] page = confluence.create_page(WIKI_SPACE, "Some title", body) ``` I've got nothing for replacing placeholders (yet), but this is the basic create-a-page-from-a-template move.

I'm having this issue on WSL, but it's because of https://github.com/skratchdot/open-golang/issues/29.

Is the default $GOBIN in your shell's $PATH? ``` ~ ❯ go help install | head -8 usage: go install [build flags] [packages] Install compiles and installs the packages named...

Line 418 is the crashing one: https://github.com/stakater/Reloader/blob/93fb083788f252ff2b7997d25a948f6a9d5f4da7/internal/pkg/handler/upgrade.go#L417-L419 I'm guessing that `containers` is an empty array, but I haven't yet figured out how that state is reached.

And now I've figured it out. `containers` is set here: https://github.com/stakater/Reloader/blob/93fb083788f252ff2b7997d25a948f6a9d5f4da7/internal/pkg/handler/upgrade.go#L387 `ContainersFunc` is [`GetRolloutContainers`](https://github.com/stakater/Reloader/blob/6b2cc9446843cb039fc795b08ab32a8fdd618e88/internal/pkg/callbacks/rolling_upgrade.go#L291-L294), which assumes the Rollout has a pod template. When a Rollout uses `workloadRef` instead, the container...

@praddy26 (or @MuneebAijaz), any thoughts on the above?