container/image id is given as nil.

[ageekymonk]

For a new_resource id is returned to be nil. So ignoring the check.
Also the -notrunc is deprecated. moving it to --no-trunc.

[amjibaly]

+1 I'm running into the same issue and this patch resolved it. Can anyone shed light on what new_resource is and under what conditions this is an issue?

[jcrobak]

this shouldn't change behavior in the case of new_resource.id being nil. In that case, container_id_matches? will always return false, and you will go into the body of the unless.

Or is my logic wrong here?

[bflad]

I think I'm going to merge this as it relates to #139, #153, and #154 and contains the -notrunc deprecation fix. Can we please verify matching works in the next release of the cookbook (0.35.0 or 1.0.0)? We can always add the additional check back in as a bugfix that needs test cases. 😉