Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

addressing issue #26 (mapping RPP and EPP as deliverable) #40

Merged
merged 4 commits into from
Nov 22, 2024

Conversation

anewton1998
Copy link
Collaborator

This PR is add mapping documents to the deliverables section. See Issue #26

Copy link
Contributor

@mwullink mwullink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

see inline comment

rpp-charter.md Outdated
@@ -64,6 +64,7 @@ After finding consensus regarding requirements, the next documents to be deliver
by this working group will describe the core architecture and JSON mapping of RPP with regard to the
provisioning of domain names, followed by any specifications necessary
to implement the core architecture and the JSON mapping for the different objects to be provisioned.
These documents may then be followed by mappings between RPP and EPP.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the "mappings between RPP and EPP" document must be moved.
new proposal:

"this working group will describe the core architecture and JSON mapping of RPP with regard to the
provisioning of domain names. These documents are then followed by a document describing the mappings between RPP and EPP. Followed by any specifications necessary to implement the core architecture and the JSON mapping for the different objects to be provisioned."

Rationale:
It may take some time (if ever) before other objects (numbers? widgets?) are described in documents, implementers need a mapping between RPP and EPP after the core architecture and JSON mapping of RPP for domains names is completed.

Copy link
Collaborator Author

@anewton1998 anewton1998 Nov 21, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How does a mapping occur before any core specifications? It seems to me that if you want to map "blah": "foo" in JSON to <blah>foo</blah> in XML, you need concrete implementation specifications.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, i don't mean to say that the mapping from RPP to EPP comes first.

i think this is the correct order:

1: the core architecture and JSON mapping of RPP with regard to the provisioning of domain names
2: mappings between RPP and EPP.
3: any other (optional) object mapping, e.g. numbers.

because at the moment it is not clear that other object mapping (3) will ever happen and if it does how long this will take. Therefore, it's better to create the mappings between RPP and EPP (2) immediately after the core architecture and JSON mapping of RPP (1)

@rpp-ietf
Copy link

rpp-ietf commented Nov 21, 2024 via email

@pawel-kow pawel-kow changed the title addressing issue #26 addressing issue #26 (mapping RPP and EPP as deliverable) Nov 21, 2024
@pawel-kow
Copy link
Collaborator

I think putting things in some order of dependencies is kind of void discussion when it comes to a charter.
The core question seems to be whether the RPP working group shall define a mapping between RPP and EPP as a deliverable.
If we say yes, then it's of secondary importance when exactly the mapping document would be ready.
The working group will likely keep it in mind already doing the architecture and defining the protocol on every level and it's enough to have a consensus that mapping is possible without yet producing a document defining it.

I would be in favour of adding such deliverable without determining the order of those now.

@mwullink
Copy link
Contributor

The wg should deliver an RPP -> EPP mapping document but i agree with Pawel that removing the ordering from the text is a good idea.

@anewton1998
Copy link
Collaborator Author

I updated this PR to remove the explicit order, instead making it an agreement with the chairs and the AD. I also moved around the paragraphs on requirements since they were intermingled with this one.

IESG documents addressing the following topics in an order determined through agreement
of the chairs with the responsible Area Director:
* core architecture
* specifications necessary to implement the core architecture
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the difference between the "core architecture" and "specifications necessary to implement the core architecture" topics?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

core architecture could be block diagrams, etc... basically words that explain how things are put together, while specifications necessary to implement them would be "send "foo": "bar" using HTTP GET`.

rpp-charter.md Outdated
Comment on lines 63 to 64
Analysis of the functionality included and commonly used in core EPP for domain names,
hosts and contacts (RFC5730,5731,5732,5733) may be created as input material for the requirements.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the rearrangement but noticed the change from "will" to "may".
The discussion on the mailing list so far was rather going into direction that such analysis will be created as an input.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If nobody is willing to do the analysis or there is no agreement on that analysis, the working group should not stall. This is an area of potential rabbit hole exploring. Thanks for noting this.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If nobody is willing to do the analysis or there is no agreement on that analysis, the working group should not stall. This is an area of potential rabbit hole exploring. Thanks for noting this.

I think if nobody would be willing to, it would indicate other problems in the working group. The conclusion of the discussion so far was that without this input we may not be able to validate if requirements and architecture are actually correct and complete to fulfil other goals and deliverables.
The text does not mean that there must be only one or more, in which form and even not if there must be a consensus around the analysis. And this is meant like this. More perspectives are typically better on an early stage.

The requirements is where we will be looking after consensus. With analysis done we may expect better quality of requirements coming out of the process.

@rpp-ietf
Copy link

rpp-ietf commented Nov 22, 2024 via email

@anewton1998
Copy link
Collaborator Author

I changed the wording to "is expected to be created". The concern is paralysis by analysis that could cause the working group to stall.

@rpp-ietf
Copy link

rpp-ietf commented Nov 22, 2024 via email

@pawel-kow
Copy link
Collaborator

I changed the wording to "is expected to be created". The concern is paralysis by analysis that could cause the working group to stall.

If you mean this wording works better to avoid this risk it is for me a good way forward. Thank you.

@pawel-kow
Copy link
Collaborator

BTW: I also kind of volunteer for the analysis work.

@anewton1998
Copy link
Collaborator Author

There is nothing stopping anybody from starting that work now. :)

@anewton1998 anewton1998 merged commit 09644c3 into SIDN:main Nov 22, 2024
@rpp-ietf
Copy link

rpp-ietf commented Nov 22, 2024 via email

@pawel-kow pawel-kow mentioned this pull request Nov 22, 2024
@rpp-ietf
Copy link

rpp-ietf commented Dec 4, 2024 via email

@rpp-ietf
Copy link

rpp-ietf commented Dec 4, 2024 via email

@mwullink
Copy link
Contributor

mwullink commented Dec 4, 2024 via email

@rpp-ietf
Copy link

rpp-ietf commented Dec 4, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants