payload: reuse supported_arches #372
Conversation
There was a problem hiding this comment.
lol, this is exactly what I was thinking about while reviewing PR that touched this file. The only cons is that the purge_previous_manifests clears all the manifests and not just the ones that will be re-created. As a result running this with both (x86_64 and s390x) and then re-running it with let's say only s390x the x86_64 manifest would be gone as well. 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 👍 |
supported_arches variable is defined but not used during manifest create step. With this patch we can easly disable arches for quick development test. Signed-off-by: Beraldo Leal <[email protected]>
beraldoleal
force-pushed
the
payload-arches
branch
from
1923189 to
5d9f0ef
Compare
supported_arches variable is defined but not used during manifest create step. With this patch we can easly disable arches for quick development test.