Not sure how easy it is to add a test to check bundled artifact from cache are dereferenced
Switch default deference back to True for tarball artifact
Review Request #4304 — Created Oct. 13, 2016 and submitted
|benjyw, jsirois, mateor, peiyu, stuhood|
https://rbcommons.com/s/twitter/r/4166 set default
deference=False, causing bundle failures because bundle cache is a link to a remote location rather than actual the content itself. This RB changes the default back to True while leaving the interface structure for other tasks to consume later on.
In this case, I wonder whether it would be easier to explicitly copy into the
What does the failure look like? It would be good to open a ticket with the error you experienced, and include it as a TODO next to
dereference=True, as I think that preserving exactly the structure of the cache entry is actually a good idea moving forward.
...to be clear: fine with the revert; just want to see a ticket. #shipit