search

confidential-containers / operator

Operator to deploy confidential containers runtime
Apache License 2.0
109 stars 58 forks source link

payload: reuse supported_arches #372

Closed beraldoleal closed 4 months ago

beraldoleal commented 5 months ago

supported_arches variable is defined but not used during manifest create step. With this patch we can easly disable arches for quick development test.

portersrc commented 5 months ago

The only cons is that the purge_previous_manifests clears all the manifests ... Anyway I don't think it's a big deal... (but it adds to the question whether we need the purge_previous_manifests at all)

purge-previous-manifests looks odd to me on multiple levels. I would handle it in a different issue and cc me on it if it's irking you, @ldoktor 👍