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

Trial and containers #4

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

evanstoner
Copy link
Contributor

No description provided.

@evanstoner evanstoner marked this pull request as draft January 6, 2025 14:28
@evanstoner evanstoner marked this pull request as ready for review January 16, 2025 15:50
@evanstoner
Copy link
Contributor Author

@solorob33 PTAL - once merged I will reorganize in the workshop studio format

@evanstoner evanstoner requested a review from solorob33 January 16, 2025 15:53
![ECR repositories](ecr-repos.png)

In the Falcon console, navigate to _Cloud security_ > _Image assessment settings_ (under _Settings_).
The ECR registry for your AWS account is already registered for image assessments. This allows the
Copy link
Collaborator

@solorob33 solorob33 Feb 3, 2025

Choose a reason for hiding this comment

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

Is the registry already registered, or are you anticipating that we'll have that already set up? Related to that, our existing code blocks pushing web-dvwa to ECR because it's assessed in pipeline and the build (ie, push to ECR) fails. To prep the demo, I manually disable the assessment, and I manually registered in ECR. That will need to be addressed. Ryan can automated ECR registration, but we'll still need to manually disable the pipeline scan which I think is a good thing, so the user gets to see the CodeBuild job setup.

Copy link
Contributor Author

@evanstoner evanstoner Feb 3, 2025

Choose a reason for hiding this comment

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

This is based purely on your demo, so if there are additional steps required we'd need to automate or explain those. I think we should keep this lab focused on the KAC prevention so we should get everything preconfigured correctly. In an extension, we can discuss shift left and show how scanning in the pipeline can prevent the image from ever being published.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants