Matt Martz

Results 336 comments of Matt Martz

I'm fine with adding this, it's simple enough. I'd go with an argument to the existing filters.

Can you use `ansible-builder create` and then using the created context, then run `buildah build` using the `Containerfile` and supplied context?

data tagging may be solving this already, since they get post validated in the TE before returning, which means we also remove all of the one off templating from `IncludedFile`....

Basically a duplicate around: * https://github.com/ansible/ansible-runner/issues/1077 * https://github.com/ansible/ansible-runner/issues/1322

In a way, I still prefer this PR over https://github.com/ansible/ansible-builder/pull/647 Even through the alternative means we have to care less, I feel like it opens us up to more issues...

> They are supposed to be the same: https://pip.pypa.io/en/stable/reference/requirement-specifiers/. Do you know of any differences? Here are a few things that work in a pip `requirements.txt` that aren't `pep508`: ```...

Closing in favor of https://github.com/ansible/ansible-builder/pull/663

In the following block, we will need to add an `except http.client.HTTPException:` with some proper error reporting: https://github.com/ansible/ansible/blob/1f987423fd4887228736f27a525a98aca36b881a/lib/ansible/modules/uri.py#L726-L734

If you have `python3-rpm` installed on your Debian host, per the documentation for `package_facts` will choose `rpm` instead of `apt` as the package manager when using `auto`: https://docs.ansible.com/ansible/latest/collections/ansible/builtin/package_facts_module.html#parameter-manager You can...

I'm not sure how to actually reproduce this. We'll need you to provide a complete, minimal reproducer. Reproducers should not use any modules or plugins that require specific environments, and...