RE: response to comments on PCELS-05/May 06
"Pana, Mircea" <[email protected]> Fri, 4 Jun 2004 16:47:56 -0400
| Newsgroups | gmane.ietf.policy |
|---|---|
| Message-ID | <[email protected]> |
To accomodate the change requested by Joel, I'm making the proposal = below. Please review it and let me know as soon as possible whether you = like or not. Instead of re-using pcimRuleValidityAssociation, PCELS would introduce a = new class: pcelsValidityAssociation. This new class would be a subclass = of pcimRuleValidityAssociation and it would not introduce any new = attributes. Its instances would be subordinated to pcelsRule instances. = (Much like pcelsConditionAssociation etc.) The pcelsRule class, instead = of reusing the pcimRuleValidityPeriodList attribute, would use a new = attribute (pcelsValidityPeriodList) as reference to its = pcelsValidityAssociation instances. In addition, Note 1 in section 5.4 would be removed. Regards, Mircea. -----Original Message----- From: Joel M. Halpern [mailto:[email protected]] Sent: Friday, June 04, 2004 9:37 AM To: [email protected] Cc: [email protected]; John Strassner; Pana, Mircea Subject: RE: response to comments on PCELS-05/May 06 I have looked this over again, and I think I understand the question. >From an object mapping and class definition perspective, it appears to = me that extending the definition of pcimRuleValidityAssociation to point = to a pcelsRule is probably not appropriate. It seems to me thinking about this that adding a different association = class for the pcelsRule - time-period relationship will not adversely = affect either existim PCLS implementors or future PCELS implementors. = Ruding churn in an I-D before publication is not a good reason to avoid = making a technically correct change. However, I could easily have missed multiple aspects of this. If there are folks looking at implementing PCELS who have an opinion on = the complexity of either the current "extension" or the proposed = additional class, please speak up. If there are LDAP folks (other than John, who has been very helpful) who = can shed light or opinions on this, I would love to hear from them. Given how many times we have been around the block on this, I would like = to ask folks to respond within one week. If we hear nothing, I will ask = Mircea and company if they can make this one last change, and hand the = document to Bert for publication. And then we will officially close the working group! Yours, Joel M. Halpern At 07:55 PM 6/3/2004 -0600, John Strassner wrote: Subject: RE: response to comments on PCELS-05/May 06 To: "Pana, Mircea" <[email protected]> Cc: <[email protected]>, <[email protected]>, <[email protected]> Hi Mircea, =20 thanks for your thoughtful response. =20 Regarding the first issue, I still disagree. The definition of the class = doesn't allow this, even though the semantics remain unchanged. Since we = are at an impasse, I'm happy to let Joel rule one way or the other. =20 Regarding changing the note in section 5.4, I agree with the change. =20 Regarding the DIT containment issue, I'm happy with your suggestion. =20 regards, John Strassner=20 -----Original Message-----=20 From: Pana, Mircea [mailto:[email protected]]=20 Sent: Friday, May 28, 2004 8:16 AM=20 To: John Strassner=20 Cc: [email protected]; [email protected]; [email protected]=20 Subject: RE: response to comments on PCELS-05/May 06 Look for my responses in <mircea4></mircea4>. Sorry, I'm slow to respond = as well.=20 Regards,=20 Mircea.=20 =20 -----Original Message-----=20 From: John Strassner [mailto:[email protected]]=20 Sent: Saturday, May 22, 2004 5:49 PM=20 To: [email protected]; John Strassner=20 Cc: [email protected]; [email protected]; [email protected]=20 Subject: RE: response to comments on PCELS-05/May 06 Please see inline, look for <js3>..</js3>. Sorry, as usual, for the = delay in responding...looks like we're down to two issues now... regards,=20 John Strassner=20 TeleManagement Forum Advisory Director John Strassner=20 Chief Strategy Officer=20 Intelliden Inc.=20 90 South Cascade Avenue=20 Colorado Springs, CO 80906 USA=20 phone: +1.719.785.0648=20 fax: +1.719.785.0644=20 email: [email protected]=20 -----Original Message-----=20 From: [email protected] [mailto:[email protected]]=20 Sent: Wednesday, May 12, 2004 5:02 PM=20 To: John Strassner=20 Cc: [email protected]; [email protected]; [email protected]=20 Subject: response to comments on PCELS-05/May 06 John, See my comments tagged <mircea3></mircea3>.=20 =20 Thank You,=20 Mircea=20 -----Original Message-----=20 From: John Strassner [mailto:[email protected]]=20 Sent: Thursday, May 06, 2004 3:44 AM=20 To: [email protected]; John Strassner=20 Cc: [email protected]; [email protected]=20 Subject: RE: your comments on PCELS-05 / April 26 I've just pulled up the remaining problems to make it easier to deal = with. I'm copying Joel and Bert so that they know that I am responding, = albeit slowly...look for <js2/>=20 =20 [...]=20 =20 Sixth, why is there no pcelsRuleValidityAssociation subclass? At this = point, <mircea>I do not understand the issue. PCELS reuses = pcimRuleValidityAssociation that is defined in PCLS </mircea> =20 <js> True, this is addressed in Note 1 in page 27 of the draft. Looking = at your class structure, since you subclassed other associations, I was = surprised that you didn't subclass this one as well. This is because = pcelsRule and pcimRule are siblings, and pcimRuleValidityPeriod (in = PCIM) is defined to exist between pcimRule and policyConditionTimePeriod = only. So, how do pcelsRule instances use a policyConditionTimePeriod? = </js>=20 <mircea2>PCELS adopts pcimRuleValidityAssociation extending its = applicability to pcelsRule. I.e. PCELS recommends the use of = pcimRuleValidityAssociation instances subordinated to pcelsRule entries = for associating pcimTPCAuxClass instances to a Rule. Note that PCELS = also recommends that: "As result, the class pcimRuleValidityAssociation = SHOULD be expected (and allowed) to have instances of pcelsRule as = superior entries." This isn't a change of semantics for the pcimRuleValidityAssociation = class. In a PCELS implementation, this class continues to represent the = PolicyRuleValidityPeriod aggregation. Other association classes defined = in PCIM are subclassed in PCELS for the purpose of extending their = semantics (and for changing their names ;-)</mircea2> <js2> The above logic is incorrect. Look at how pcimRuleValidityPeriod = is defined in RFC3060. It is NOT a representation of a = PolicyRuleValidityPeriod, it is a representation of the *association* = between a PolicyRuleValidityPeriod and a PolicyRule. So, the problem is = that you have defined a pcelsPolicySet to be a sibling of pcimRule (to = which pcimRuleValidityPeriod applies). Therefore, pcimRuleValidityPeriod = does NOT apply to pcelsPolicySet (and thus not to pcelsRule). Therefore, = you have to either redefine this association, or create a new one. You = did neither. </js2>=20 <mircea3>I must be missing something... You say that = pcimRuleValidityPeriod is defined in RFC3060 but in PCIM I can not find = any reference to pcimRuleValidityPeriod. PCLS (RFC3703) does not define = such class either. You also mention the association between a = PolicyRuleValidityPeriod and a PolicyRule but I can not find anything in = either PCIM or PCLS that implements such association.=20 <js3> Technically correct, but we need to use a little imagination here = ;-) Of course if you search for that string it won't appear because the = RFC doesn't use prefixes. This maps to "PolicyRuleValidityPeriod" which = is defined in Section 7.7, page 51. And if you read this section, it = defines PolicyRuleValidityPeriod as an aggregation. </js>=20 However, in PCIM I read that PolicyRuleValidityPeriod is defined as "A = class representing the aggregation of PolicyTimePeriodConditions by a = PolicyRule". In PCLS I read that "The policyRuleValidityPeriod = aggregation is mapped to the PCLS pcimRuleValidityAssociation class." = So, I conclude that pcimRuleValidityAssociation is used to associate = TimePeriodConditions to a Rule. For PCLS this means associating = instances of pcimTPCAuxClass to a pcimRule. For PCELS this would mean = associating instances of pcimTPCAuxClass to a pcelsRule. I really don't = see the problem. </mircea3>=20 <js3> Huh? Look at your class hierarchy. And look at your reply in = Mircea2 ("...extending [sic] its applicability to pcelsRule.") That's = fine for a fluffy spec, but this is a spec of a DATA MODEL, and one must = not make such assumptions. Strictly speaking, since they are siblings, = this aggregation does NOT apply to pcelsRule. Therefore, you must either = define a new association (preferred) or change something). Note also that I completely disagre with your logic saying that "this = and this SHOULD happen". You (well, PCIMe) altered the picture in the = definition of these classes, and we can't simply wave our hands saying = that this should happen by magic without specifying it rigorously in the = data model. </js>=20 <mircea4>I think that I understand the issue now: You are arguing that, = because pcimRuleValidityAssociation is defined to be associated (and = subordinated) to a pcimRule, a submodel is NOT allowed to use it = anywhere else even if its semantics (i.e. PolicyRuleValidityPeriod) are = preserved. This is somewhat similar to our previous argument about LDAP = attribute type re-use. It is, indeed, "elegant" to define new = classes/attributes as soon as the slightest context change is = introduced. However, in order to reduce the churn (and this has been requested by = PCELS reviewers) I have a strong preference for the current proposal. I = believe that it is the most *practical* approach to the problem. Wrt. the text at the end of note 1 in section 5.4 of PCELS = ("pcimRuleValidityAssociation SHOULD be expected..."), for clarity, I = suggest changing it to: "If DIT structure rules and name forms are = written for a PCELS implementation (as suggested in section 5.5 of = [PCLS]), they would require that an instance of the = pcimRuleValidityAssociation class have as its superior an instance of = the pcelsRule class or, if applicable, an instance of the pcimRule = class. Any structure rules and name forms that require an instance of = the pcimRuleValidityAssociation class to have as its superior only an = instance of the pcimRule class, are in conflict and MUST be = removed."</mircea4>