On 11/6/15 2:46 PM, Mike McLean wrote:
On 11/06/2015 04:49 PM, Mike Bonnet wrote:
> The attached patch causes Maven builds to fail if they produce a file
> type that Koji doesn't know about. This is preferable to silently
> ignoring them.
I had sort of assumed the code was already doing this. It does for
images, and for manual imports (the latter because we run
get_archive_type with strict on).
So, we're going from silently ignoring a condition to failing on it. Do
we have any sense of how often this happens?
It has happened a couple times recently, with undesirable results. The
affected teams are the ones requesting this change.
Thinking further down the road, this really seems like the wrong place
to have this check. I'm not sure the hub should really care about this.
Also, seems silly for the builder to bother uploading content that's
going to be rejected. ...maybe not something we're going to manage to
fix in 1.x though
There are some good reasons to upload content which we don't want in the
database. For example, for Maven builds we don't import
maven-metadata.xml (it's regen'ed on a per-repo basis because it
aggregates information from multiple builds) but we want it in the task
output directory because that directory is then a valid Maven repo,
which we point subsequent tasks at in a maven-chain build.
While we're at it should we also make the same change in
completeWinBuild?
Agreed, updated patch attached.
_______________________________________________
koji-devel mailing list
koji-devel(a)lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/koji-devel