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

Approval Status and Date for OSCAL Document Instance #1033

Closed
6 tasks done
ohsh6o opened this issue Oct 4, 2021 · 6 comments · Fixed by #1052 or #1429
Closed
6 tasks done

Approval Status and Date for OSCAL Document Instance #1033

ohsh6o opened this issue Oct 4, 2021 · 6 comments · Fixed by #1052 or #1429
Assignees
Labels
Discussion Needed This issues needs to be reviewed by the OSCAL development team. enhancement Model Engineering An issue to be discussed during the bi-weekly Model Engineering Meeting User Story
Milestone

Comments

@ohsh6o
Copy link
Contributor

ohsh6o commented Oct 4, 2021

User Story:

As an OSCAL tool developer, in order to allow my tools to show that different stakeholders from the organization authoring the information system's security documentation (SSP, SAP, SAR, and/or POA&M), I want to add metadata to show the responsible parties for approval and dates of these approvals.

Goals:

@david-waltermire-nist and I had found a way to effectively encode only the responsible party for these approvals, but current use of additional props, existing OSCAL generic metadata structures, and/or specific structures for particular OSCAL models cannot effectively encode and map an approval date for each responsible party. There is a strong likelihood there will be more than one approval party and approval date in most cases.

See GSA/fedramp-automation#162 for more context.

Dependencies:

N/A

Acceptance Criteria

  • All OSCAL website and readme documentation affected by the changes in this issue have been updated. Changes to the OSCAL website can be made in the docs/content directory of your branch.
  • A Pull Request (PR) is submitted that fully addresses the goals of this User Story. This issue is referenced in the PR.
  • The CI-CD build process runs without any reported errors on the PR. This can be confirmed by reviewing that all checks have passed in the PR.

{The items above are general acceptance criteria for all User Stories. Please describe anything else that must be completed for this issue to be considered resolved.}

@ohsh6o
Copy link
Contributor Author

ohsh6o commented Oct 14, 2021

Per conversation today with Dave and Wendell, I am going to make a recommended architecture design a new assembly. It will look something like this.

  • An approvals assembly
    • With 0 to unbounded instances of an approval. Each approval will have:
      • party
      • role
      • date
      • remarks
      • links
      • prop

Will draft a PR with proposed changes to the Metaschema for review, hopefully for tomorrow's model review meeting.

@xee5ch
Copy link
Contributor

xee5ch commented Nov 10, 2021

As @ohsh6o is inactive, I will try and pick this up. I was talking to @wendellpiez and think, given feedback from the model meeting and his recommendation, I will try to update this PR and convert approvals and approval to actions and action.

@aj-stein-nist
Copy link
Contributor

Whew, this guy! Per discussion with Dave, I like the previously implemented assembly designs, I still want to talk over potential naming collisions and other stuff prior to merging this, given Dave and team's approval.

Meeting invite sent for next week.

@aj-stein-nist aj-stein-nist moved this from Todo to In Progress in NIST OSCAL Work Board Aug 10, 2022
@aj-stein-nist aj-stein-nist moved this from In Progress to Under Review in NIST OSCAL Work Board Aug 11, 2022
@aj-stein-nist
Copy link
Contributor

OK the contributed PR is ready for review and should be good to go for Dave. Sliding this to Under Review.

@david-waltermire
Copy link
Contributor

PR #1052 is ready to merge. Need community review.

@david-waltermire david-waltermire added Discussion Needed This issues needs to be reviewed by the OSCAL development team. Model Engineering An issue to be discussed during the bi-weekly Model Engineering Meeting labels Aug 25, 2022
@david-waltermire david-waltermire moved this from Under Review to Reviewer Approved in NIST OSCAL Work Board Aug 25, 2022
@david-waltermire david-waltermire linked a pull request Aug 26, 2022 that will close this issue
8 tasks
@david-waltermire
Copy link
Contributor

Going to merge #1429 since there has been no additional community feedback.

Repository owner moved this from Reviewer Approved to Done in NIST OSCAL Work Board Sep 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Discussion Needed This issues needs to be reviewed by the OSCAL development team. enhancement Model Engineering An issue to be discussed during the bi-weekly Model Engineering Meeting User Story
Projects
Status: Done
4 participants