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

Advanced Class updates #275

Merged
merged 23 commits into from
Dec 5, 2024
Merged

Advanced Class updates #275

merged 23 commits into from
Dec 5, 2024

Conversation

wdower
Copy link
Contributor

@wdower wdower commented Dec 4, 2024

TODO

  • review all pages for typos/broken links
  • update the first example resource that we tend to skip anyway
  • add an appendix about InSpec parallel

Copy link

netlify bot commented Dec 4, 2024

Deploy Preview for mitre-saf-training ready!

Name Link
🔨 Latest commit 29b25b3
🔍 Latest deploy log https://app.netlify.com/sites/mitre-saf-training/deploys/6751bb95d5ebcd0008a35f77
😎 Deploy Preview https://deploy-preview-275--mitre-saf-training.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 73 (🔴 down 19 from production)
Accessibility: 97 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

src/courses/advanced/README.md Outdated Show resolved Hide resolved
- Develop new InSpec resources to query new types of targets or components.
- Know how to propose a pull request to Chef InSpec to contribute your improved/developed InSpec - resources back to the community.
- Propose a pull request to Chef InSpec to contribute your improved/developed InSpec resources back to the community.

## 1.2 About InSpec
Copy link
Contributor

Choose a reason for hiding this comment

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

we should make sure that the beginner course and the advanced course share the same bullet points here. there are several textual differences here

Copy link
Contributor

Choose a reason for hiding this comment

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

@Amndeep7 confused about why the objectives for beginner and advanced should be the same

Copy link
Contributor

Choose a reason for hiding this comment

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

@karikarshivani I was talking about the "About InSpec" section

src/courses/advanced/README.md Show resolved Hide resolved
src/courses/advanced/README.md Show resolved Hide resolved
src/courses/advanced/README.md Show resolved Hide resolved
src/courses/advanced/12.md Show resolved Hide resolved
src/courses/advanced/12.md Show resolved Hide resolved

### Reference Other Class Content

This class is part of a set of security automation content offered by the MITRE SAF(c) team. If you found this content interesting and want to learn more, we encourage you to explore the User Class or Beginner Security Automation Developer Class, as shown in the table of contents on the left.
Copy link
Contributor

Choose a reason for hiding this comment

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

Talk to Aaron but I think we need to be using tm not c


### Reference Other Class Content

This class is part of a set of security automation content offered by the MITRE SAF(c) team. If you found this content interesting and want to learn more, we encourage you to explore the User Class or Beginner Security Automation Developer Class, as shown in the table of contents on the left.
Copy link
Contributor

Choose a reason for hiding this comment

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

based off of how we teach this class, they usually will have already gone through the user and beginner classes. we also reference them heavily here. i think we should point people at the guidance course and maybe aaron's fancy notes of how to really develop a profile.

@@ -5,20 +5,20 @@ author: Aaron Lippold
headerDepth: 3
---

### RHEL7 Pipeline example
### RHEL7 Pipeline Example
Copy link
Contributor

Choose a reason for hiding this comment

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

Non-blocker - we should update this to a more recent rhel

p-oneil and others added 3 commits December 5, 2024 07:29
Co-authored-by: Amndeep Singh Mann <[email protected]>
Signed-off-by: Shivani Karikar <[email protected]>
Signed-off-by: Shivani Karikar <[email protected]>
Co-authored-by: Amndeep Singh Mann <[email protected]>
Signed-off-by: Shivani Karikar <[email protected]>
wdower and others added 2 commits December 5, 2024 07:48
Signed-off-by: Will <[email protected]>
Co-authored-by: Amndeep Singh Mann <[email protected]>
Signed-off-by: Shivani Karikar <[email protected]>
@karikarshivani karikarshivani merged commit 31e80c0 into main Dec 5, 2024
9 checks passed
@karikarshivani karikarshivani deleted the dec24-advanced branch December 5, 2024 14:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants