search

panrg / path-properties

A Vocabulary of Path Properties
Other
1 stars 3 forks source link

Add target property definition (fixes #69) #77

Closed cyrill-k closed 2 years ago

cyrill-k commented 2 years ago

Added a new definition Target property and rewrote section 3.1 to refer to this new term (see #69). Also added a comparison to SLE, SLI, and SLOs with a reference to the IETF network slices draft.

cyrill-k commented 2 years ago

Thanks for the textual improvements! I committed your suggestions and added two suggestions based on your feedback.

Regarding the merge of texts relating to the usefulness of path properties discussed in #75, we could merge the paragraph talking about the property trade-offs ("Note, there may be trade-offs between path properties ...") with the first paragraph: "... may lead to unintended feedback loops. Also, there may be trade-offs between path properties ..." What do you think?

renghardt commented 2 years ago

Regarding the merge of texts relating to the usefulness of path properties discussed in #75, we could merge the paragraph talking about the property trade-offs ("Note, there may be trade-offs between path properties ...") with the first paragraph: "... may lead to unintended feedback loops. Also, there may be trade-offs between path properties ..." What do you think?

I agree that these should be merged.

To me, it would make most sense to move the text in the first paragraph (starting with "Care needs to be taken […]" and ending with "[…] may lead to unintended feedback loops.") a bit down and then embed it there. That way, we'd first discuss the general concepts around objectives and target properties, and then have the caveats all in one place later. I will make a suggestion.