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

Data Center constraints do not conform with upstream OSCAL expectations #818

Closed
1 task done
kyhu65867 opened this issue Oct 24, 2024 · 2 comments · Fixed by #819 or GSA/automate.fedramp.gov#96
Closed
1 task done
Assignees
Labels
bug Something isn't working

Comments

@kyhu65867
Copy link

kyhu65867 commented Oct 24, 2024

This relates to ...

  • the FedRAMP OSCAL Validations

What happened?

When @aj-stein-gsa changed "name" to a value of "type" in the ssp-ALL-VALID file, test cases for the data center constraints broke. This is because the constraints are written to expect a "name" value of "data-center".

Common practice is for the name field to equal "type" and for the value field to equal "data-center". Thus, Constraints and test cases need to be adjusted accordingly.
Screenshot 2024-10-24 at 11 41 07 AM

Relevant log output

No response

How do we replicate this issue?

Change ssp-all-VALID to the correct syntax and watch the test harness fail.

Where, exactly?

  • Changes need to be made in ssp-all-valid to make it conformant.
  • Subsequently, changes need to be made in fedramp-external-constraints to the data center related constraints
  • Finally, changes need to be made to the unit tests for these constraints

Other relevant details

Another issue I noticed is that the unit tests are split up into multiple files, like "ssp-data-center-alternate-INVALID", but there are multiple things wrong in that file, AKA the country code is also incorrect. I think this defeats the purpose of unit tests, because how do we know if it is invalid because the alternate is invalid if there are multiple issues appearing? I will try to address this but I don't know how widespread of a problem this is

@kyhu65867 kyhu65867 added the bug Something isn't working label Oct 24, 2024
@kyhu65867 kyhu65867 self-assigned this Oct 24, 2024
@aj-stein-gsa aj-stein-gsa moved this from 🆕 New to 📋 Backlog in FedRAMP Automation Oct 24, 2024
@aj-stein-gsa aj-stein-gsa linked a pull request Oct 24, 2024 that will close this issue
6 tasks
@aj-stein-gsa aj-stein-gsa moved this from 📋 Backlog to 🏗 In progress in FedRAMP Automation Oct 24, 2024
@aj-stein-gsa aj-stein-gsa moved this from 🏗 In progress to 👀 In review in FedRAMP Automation Oct 24, 2024
@aj-stein-gsa aj-stein-gsa moved this from 👀 In review to 🏗 In progress in FedRAMP Automation Oct 24, 2024
@aj-stein-gsa
Copy link
Contributor

We have some minor fixes to add, so I will move this back to in progress for @kyhu65867 to address and then we can come back to it quickly.

@aj-stein-gsa aj-stein-gsa linked a pull request Oct 25, 2024 that will close this issue
@aj-stein-gsa aj-stein-gsa moved this from 👀 In review to 🛑 Blocked in FedRAMP Automation Oct 25, 2024
@aj-stein-gsa
Copy link
Contributor

This can move from blocked to in review and ready to ship once GSA/automate.fedramp.gov#96 is merged.

@aj-stein-gsa aj-stein-gsa moved this from 🛑 Blocked to 👀 In review in FedRAMP Automation Oct 25, 2024
@aj-stein-gsa aj-stein-gsa moved this from 👀 In review to 🚢 Ready to Ship in FedRAMP Automation Oct 25, 2024
@github-project-automation github-project-automation bot moved this from 🚢 Ready to Ship to ✅ Done in FedRAMP Automation Oct 25, 2024
@aj-stein-gsa aj-stein-gsa moved this from ✅ Done to 🚢 Ready to Ship in FedRAMP Automation Oct 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
2 participants