Catch and log errors in @Observes methods, but do not fail the suite and allow cleanups - This should actually be the default for any @Observes method, as the CDI events are synchronous - A failure prevents other observers from running, such as cleanups etc.
Also, the Cube internal images might need to be pulled from a user specified docker registry. So this PR also adds and new "dockerRegistry" property - I did think about using the docker qualifier, but using cubedrone allows for a different registry if required (actually have a use case for this).
Changes proposed in this pull request:
Added try/catch/log to @Observes methods in above classes
Short description of what this resolves:
Catch and log errors in
@Observes
methods, but do not fail the suite and allow cleanups - This should actually be the default for any@Observes
method, as the CDI events are synchronous - A failure prevents other observers from running, such as cleanups etc.Also, the Cube internal images might need to be pulled from a user specified docker registry. So this PR also adds and new "dockerRegistry" property - I did think about using the docker qualifier, but using cubedrone allows for a different registry if required (actually have a use case for this).
Changes proposed in this pull request:
@Observes
methods in above classesFixes #1098