Re: Prop expansion via fileinstall but not configuration admin

On Sat, Sep 17, 2016 at 1:27 PM, David Jencks wrote: > >> On Sep 17, 2016, at 10:16 AM, Benson Margulies wrote: >> >> David, the only case I was concerned with is when there is a >> pre-existing file with the syntax of my test case, which is

Re: Prop expansion via fileinstall but not configuration admin

p.s. It's always possible that I'm _wrong_, and CA is logging, but I didn't have the log service set up soon enough. That's another reason for me to be the one to spend time trying to make a test case. On Sat, Sep 17, 2016 at 1:16 PM, Benson Margulies wrote: > David, the

Re: Prop expansion via fileinstall but not configuration admin

Along with Carsten, I’m confused by your jira report. What happens when you create a configuration in code and try to update it with your property? Can you get it back? There might be a problem with persisting such a property, but your test doesn’t demonstrate it because perhaps the

Re: Prop expansion via fileinstall but not configuration admin

I could commit this test, which fails, if you like, with an @Ignore. public class InvalidFileSyntaxTest { private static final String CONFIG = "rootPath=${bt.root}\n"; @Test public void testPropSyntax() throws IOException { final Dictionary dict =

Re: Prop expansion via fileinstall but not configuration admin

On Sat, Sep 17, 2016 at 12:45 PM, David Jencks wrote: > >> On Sep 17, 2016, at 9:40 AM, Benson Margulies wrote: >> >> On Sat, Sep 17, 2016 at 3:21 AM, Carsten Ziegeler > > wrote: I

Re: Prop expansion via fileinstall but not configuration admin

> On Sep 17, 2016, at 9:40 AM, Benson Margulies wrote: > > On Sat, Sep 17, 2016 at 3:21 AM, Carsten Ziegeler > wrote: >>> I don’t think this belongs in ca. You could use a ConfigurationPlugin. >>> Unfortunately

Re: Prop expansion via fileinstall but not configuration admin

On Sat, Sep 17, 2016 at 3:21 AM, Carsten Ziegeler wrote: >> I don’t think this belongs in ca. You could use a ConfigurationPlugin. >> Unfortunately you’ll have to wait till R7 until this works with DS. Maybe >> Carsten already implemented the CA part, but I didn’t do

Re: Prop expansion via fileinstall but not configuration admin

> I don’t think this belongs in ca. You could use a ConfigurationPlugin. > Unfortunately you’ll have to wait till R7 until this works with DS. Maybe > Carsten already implemented the CA part, but I didn’t do the DS part yet. > I've implemented both parts, the CA part is in trunk, the DS

Re: Prop expansion via fileinstall but not configuration admin

I don’t think this belongs in ca. You could use a ConfigurationPlugin. Unfortunately you’ll have to wait till R7 until this works with DS. Maybe Carsten already implemented the CA part, but I didn’t do the DS part yet. Alternatively you can code it into whatever management agent you are

Prop expansion via fileinstall but not configuration admin

The system property expansion feature of the configuration-admin behavior of fileinstall is quite convenient. I could code it, optionally, into confadmin. I wish I could have it without all the other mechanism of fileinstall that I don't need. Acceptable?