@mikem I would like to completely drop first part of condition and create arch dir for all tasks (even for image). Do you think it is ok?

I would like to completely drop first part of condition and create arch dir for all tasks (even for image). Do you think it is ok?

I was briefly concerned that there could be a case of external code relying on importImage and not providing the arch field, but it appears that this is explicitly required in the docstring, and relied upon in the underlying import_image() function. So I think it's ok to assume that it's there.

It looks like get_build_logs will handle this fine. It doesn't care what the log layout it; it just walks the logdir and reports the logs it finds.

From a purely Koji pespective, I think this is ok.

My only remaining concern is that CI systems could conceivably be looking at particular paths for logs. This is technically a backwards incompatible change, though certainly arguably better.

Is there a reason for dropping the first condition besides making the code a little smaller? Unless there is, I guess I'd err on the side of just merging this as-is.

No, it is just consistence/cleanup. So, let's gowith merge as is.

Commit bc047089 fixes this pull-request