[Pki-devel] [PATCH] 376 Fixed TPS resource statuses.
Endi Sukma Dewata
edewata at redhat.com
Thu Jan 30 15:49:28 UTC 2014
On 1/28/2014 7:47 PM, John Magne wrote:
> ACK:
Thanks. Pushed to master.
> A few simple questions.
>
> 1. I notice we have resources for CS.cfg Connections. Does this clash with the so-called "Connectors"
> we already have in the Java subsystems? Or are they designed as a replacement? I'm speaking of the
> Connectors that allow us to talk from subsystem to subsystem.
Do you mean the KRAConnector in CA and TPSConnector in TKS? It looks
like they are designed for different purpose. They use specific
parameters which don't match with the parameters used in TPS. The
Connections in TPS are used to manage generic connection configurations
in TPS. It doesn't actually establish a connection yet.
> 2. I notice we have separate classes for the various types of CSCfgRecords, being ConnectionRecord, or ProfileRecord, etc.
> The entirety of those classes seem to be taken care of by the base class, CSCfgRecord.
Yes. They are originally unrelated classes but now they inherit from the
same base class.
> Further down the line we have XXXDatabase classes that have methods to add these records, such as CreateConnectionRecord, or CreateProfileRecord.
> The methods appear to be pretty similar to each other. Would it have been possible to put much of this down into the base class?
These classes have been simplified quite a bit. I don't want to simplify
too much in case there are resource-specific logic that has to be added
while I'm still looking at the original TPS code, but we can certainly
clean it up later.
--
Endi S. Dewata
More information about the Pki-devel
mailing list