Kevin W Monroe

One other charm definition I tried was to remove the `cs:` prefix: ``` prometheus: charm: '~prometheus-charmers/xenial/prometheus' series: xenial num_units: 1 ``` But was still met with an `entity_url` that didn't...

Aha! Using the revno makes it go real good: ``` prometheus: charm: 'cs:xenial/prometheus-5' ``` Looks like without a rev, `deploy_service` falls back to whatever comes out of `CharmStoreID(info["Id"])` https://github.com/conjure-up/conjure-up/blob/master/conjureup/juju.py#L622 It'd...

> The out-of-the-box bundle merging has the fragments' machines section entirely replace the base bundle's (this seems very wrong) Here's a paste supporting the above observation with juju-2.3-rc2: http://paste.ubuntu.com/26060584/. I'll...

I think this is related to #1232. I worked around it in that issue by being explicit with the revno. Is it possible you ran into this by specifying the...

+1, LGTM. Fwiw, i was getting the same error (failsafe-summary.xml: no such file) with v2.17, 2.18, and 2.18.1. I didn't dig too deep, but this fits the timeframe and seems...

Hey @evans-ye, I see this has been committed upstream. Just FYI, if you included a "Closes #244" in the upstream commit message, the github bot would have automatically closed this...

@evans-ye while you're updating spark's init.pp, would you consider replacing the `spark-datanucleus` package definition with an `include spark::datanucleus`? https://github.com/evans-ye/bigtop/blob/aeb72aa14ecc3363723dce542f631aea2faaba41/bigtop-deploy/puppet/modules/spark/manifests/init.pp#L158 As you noted in BIGTOP-2764, including the class seems better than...

One other thing @buggtb, since this opens a new port, we'll need 10000 listed in layer.yaml: https://github.com/apache/bigtop/blob/master/bigtop-packages/src/charm/spark/layer-spark/layer.yaml#L25 As well as a check for `spark_enable_thriftserver` in the `start` method to know...

@johnsca cool, thanks. I will prep a PR to include `-m`.

Fixed in #134 with https://github.com/conjure-up/spells/pull/134/commits/e5998344ebe81f516ab24cab8ff2b23f06712781