Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-674

Adopt new document LDM-732

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:

      Description

      New document LDM-732

      Vera C. Rubin Network Verification Document:

      is ready to be approved.

      The document preview is available at the following link:

      https://ldm-732.lsst.io/v/RFC-674/index.html

       

      Comments can be done on the following PR:

      https://github.com/lsst/LDM-732/pull/2

        Attachments

          Issue Links

            Activity

            No builds found.
            gcomoretto Gabriele Comoretto [X] (Inactive) created issue -
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Field Original Value New Value
            Description New document LDM-732

            Vera C. Rubin Network Verification Document:

            is ready to be approved.
            New document LDM-732

            Vera C. Rubin Network Verification Document:

            is ready to be approved.

            The document preview is available at the following link:

            [https://ldm-732.lsst.io/v/RFC-674/index.html]

             

            Comments can be done on the following PR:

            [https://github.com/lsst/LDM-732/pull/2]
            tjenness Tim Jenness made changes -
            Status Proposed [ 10805 ] Flagged [ 10606 ]
            tjenness Tim Jenness made changes -
            Watchers Gabriele Comoretto [ Gabriele Comoretto ] Colin Slater, Gabriele Comoretto, Jeff Kantor, John Swinbank, Kian-Tat Lim, Leanne Guy, Michelle Butler, Tim Jenness, Wil O'Mullane [ Colin Slater, Gabriele Comoretto, Jeff Kantor, John Swinbank, Kian-Tat Lim, Leanne Guy, Michelle Butler, Tim Jenness, Wil O'Mullane ]
            tjenness Tim Jenness made changes -
            Labels LDM-732
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 23410 ]
            Hide
            womullan Wil O'Mullane added a comment -

            No problem with this - not obvious we need to fla git but it I suppose it is a new Change Control Document.

            Show
            womullan Wil O'Mullane added a comment - No problem with this - not obvious we need to fla git but it I suppose it is a new Change Control Document.
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 23499 ]
            Hide
            tjenness Tim Jenness added a comment -

            Leanne Guy has agreed to review this before we approve.

            Show
            tjenness Tim Jenness added a comment - Leanne Guy has agreed to review this before we approve.
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 23736 ]
            Hide
            lguy Leanne Guy added a comment -

            Looks good. I have some requests for changes.

            1) This is a verification document and should not include details of test cases associated with the verification elements. All sub-sub-sections of section 2 sub-sections should be removed and instead, just a list of the associated test cases (as per the 'test Case Summary'), with perhaps the test case objective added. The predecessors, prerequisites and test personnel are not appropriate in a verification document, and belong instead in the test specification

            2) The objective of many of the test cases included is not an objective but rather is a detailed description of how to execute the test case. All test case objectives should be rewritten to be short clear statements of the objective and the details of how to execute the test moved to the test script. E.g 2.4.1 [LVV-T186] Objective should be reduced to something like: "Test the summit to base network reliability without physically cutting network", and then all the other details moved to the test script

            Show
            lguy Leanne Guy added a comment - Looks good. I have some requests for changes. 1) This is a verification document and should not include details of test cases associated with the verification elements. All sub-sub-sections of section 2 sub-sections should be removed and instead, just a list of the associated test cases (as per the 'test Case Summary'), with perhaps the test case objective added. The predecessors, prerequisites and test personnel are not appropriate in a verification document, and belong instead in the test specification 2) The objective of many of the test cases included is not an objective but rather is a detailed description of how to execute the test case. All test case objectives should be rewritten to be short clear statements of the objective and the details of how to execute the test moved to the test script. E.g 2.4.1 [LVV-T186] Objective should be reduced to something like: "Test the summit to base network reliability without physically cutting network", and then all the other details moved to the test script
            Hide
            jkantor Jeff Kantor added a comment -

            After the Network Engineering Team provides their final comments (sometime after April 9 NET meeting) I will incorporate those changes and Gabrielel and I will make the recommended restructuring.

            Show
            jkantor Jeff Kantor added a comment - After the Network Engineering Team provides their final comments (sometime after April 9 NET meeting) I will incorporate those changes and Gabrielel and I will make the recommended restructuring.
            Hide
            lguy Leanne Guy added a comment -

            Thank you

            Show
            lguy Leanne Guy added a comment - Thank you
            lguy Leanne Guy made changes -
            Planned End 03/Mar/20 6:57 PM 17/Mar/20 6:57 PM
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 23829 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 23911 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 23981 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 24080 ]
            Hide
            gcomoretto Gabriele Comoretto [X] (Inactive) added a comment -

            Leanne Guy, LDM-732 has been updated accordingly. I update the generation tool in order to include in the document only the relevant VE information. Jeff Kantor updated the test cases content.

            Please have a look to https://ldm-732.lsst.io/v/RFC-674/index.html and let me know if OK.

            Show
            gcomoretto Gabriele Comoretto [X] (Inactive) added a comment - Leanne Guy , LDM-732 has been updated accordingly. I update the generation tool in order to include in the document only the relevant VE information. Jeff Kantor updated the test cases content. Please have a look to  https://ldm-732.lsst.io/v/RFC-674/index.html  and let me know if OK.
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 24228 ]
            Hide
            lguy Leanne Guy added a comment -

            The first point has been addressed satisfactorily. Concerning point 2, I have added some comments in the PR that should be addressed.

            Show
            lguy Leanne Guy added a comment - The first point has been addressed satisfactorily. Concerning point 2, I have added some comments in the PR that should be addressed.
            Hide
            gcomoretto Gabriele Comoretto [X] (Inactive) added a comment -

            I think I have addressed all comments. Please see:

            https://ldm-732.lsst.io/v/RFC-674/index.html

            I hope we are ready to go now.

             

            Show
            gcomoretto Gabriele Comoretto [X] (Inactive) added a comment - I think I have addressed all comments. Please see: https://ldm-732.lsst.io/v/RFC-674/index.html I hope we are ready to go now.  
            Hide
            lguy Leanne Guy added a comment -

            Thanks - all comments addressed now

            Show
            lguy Leanne Guy added a comment - Thanks - all comments addressed now
            Hide
            jkantor Jeff Kantor added a comment -

            Thanks Leanne! And many thanks to Gabrielle for all his support!

            Show
            jkantor Jeff Kantor added a comment - Thanks Leanne! And many thanks to Gabrielle for all his support!
            lguy Leanne Guy made changes -
            Labels LDM-732 LDM-732 dm-sst
            Hide
            tjenness Tim Jenness added a comment -

            It sounds like we can now board recommend this.

            Show
            tjenness Tim Jenness added a comment - It sounds like we can now board recommend this.
            lguy Leanne Guy made changes -
            Status Flagged [ 10606 ] Board Recommended [ 11405 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Status Board Recommended [ 11405 ] Adopted [ 10806 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Resolution Done [ 10000 ]
            Status Adopted [ 10806 ] Implemented [ 11105 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 24321 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 24382 ]
            lguy Leanne Guy made changes -
            Labels LDM-732 dm-sst DM-SST LDM-732 dm-sst
            lguy Leanne Guy made changes -
            Labels DM-SST LDM-732 dm-sst DM-SST LDM-732

              People

              Assignee:
              gcomoretto Gabriele Comoretto [X] (Inactive)
              Reporter:
              gcomoretto Gabriele Comoretto [X] (Inactive)
              Watchers:
              Colin Slater, Gabriele Comoretto [X] (Inactive), Jeff Kantor, Jeffrey Carlin, John Swinbank, Kian-Tat Lim, Leanne Guy, Michelle Butler [X] (Inactive), Tim Jenness, Wil O'Mullane
              Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.