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

Automate File Type Conversion Checks #129

Closed
2 of 4 tasks
ohsh6o opened this issue Jul 6, 2021 · 3 comments
Closed
2 of 4 tasks

Automate File Type Conversion Checks #129

ohsh6o opened this issue Jul 6, 2021 · 3 comments
Labels
bug Something isn't working

Comments

@ohsh6o
Copy link
Contributor

ohsh6o commented Jul 6, 2021

Action Item

This is a ...

  • fix - Something needs to be different.

This relates to ...

  • Other

Describe the problem or enhancement

This is a follow-on for ongoing development effort for the NIST OSCAL CI/CD to address ongoing work that was not complete for #103.

Goals:

When common document instances are converted between XML and JSON and YAML in the NIST OSCAL CI/CD system, file paths are presumed to be similar and then extensions are switched out by a perl-based regex that does the file extension substitution. This has broken for a period, which led to the linked issue and relevant issue for the NIST OSCAL content repo, usnistgov/oscal-content#59.

Dependencies:

None

Acceptance Criteria

  • All FedRAMP Documents Related to OSCAL Adoption affected by the changes in this issue have been updated.
  • A Pull Request (PR) is submitted that fully addresses the goals of this User Story. This issue is referenced in the PR.
@ohsh6o
Copy link
Contributor Author

ohsh6o commented Jul 26, 2021

FYI, @rgauss since it is a source of all the kludges in the repo fork you maintain: I have made some headway on this effort so that you and all other NIST OSCAL/FedRAMP consumers can make use of OSCAL documents, hopefully, and get converted file-types and not have to work around it. I will require adding in a Python script to the CI/CD processing, and it turned out a little more complicated than I planned. Simply switching out xml for json is not always ideal and I went down a rabbit hole to find a structured way to resolve it. More to follow!

@volpet2014
Copy link
Contributor

This will be resolved when FedRAMP Automation #241 is closed.

@aj-stein-gsa
Copy link
Contributor

This issue seems very likely easy to close given the changes in code and infrastructure in ... wow ... more than three years, but I need to double check what we will do with these paths and files moving forward with #738. Will close this for now.

By old @ohsh6o AJ. 👋

@aj-stein-gsa aj-stein-gsa closed this as not planned Won't fix, can't repro, duplicate, stale Oct 25, 2024
@github-project-automation github-project-automation bot moved this from 📋 Backlog to ✅ Done 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
Development

No branches or pull requests

4 participants