Re: [libvirt] [PATCH 01/10] secret: Move virSecretObj to secret_conf.h

On 03/07/2016 10:50 AM, Peter Krempa wrote: > On Wed, Mar 02, 2016 at 13:54:58 -0500, John Ferlan wrote: >> To support being able to create a hashed secrets list, move the >> virSecretObj to secret_conf.h so that secret_conf.c can at least find >> the definition. > > I don't think this is necessary. You still can create accessors and have > the actual implementation of the struct private in the .c file. That way > it's guaranteed that nobody will touch the fields directly accross the > source files. > Suffice to say there's still more work to do; however, short term I think it needs to move so I can make some progress. I will have a followup series that will move _virSecretObj into secret_conf.c. The goal for this series was to build up the code to support a hashed secret object while still having that linked list in the driver. Patch 10 was added to this series mainly since it was discussed in danpb's review of my last set of secret driver changes. Moving the saving of the config file and secret data file was next on the todo list. I should note that the existing model has virDomainObj in src/conf/domain_conf.h and virNetworkObj in src/conf/network_conf.h, so virSecretObj wouldn't be unique...

I honestly didn't want another 15-20 patch series floating around. Make some progress, share, get buy in, continue. The question then becomes does the rest of the series look good?