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

feat(terraform): include pods of kubernetes_deployment in kubernetes_pod checks (4/4) #3738

Merged
merged 1 commit into from
Nov 7, 2022

Conversation

ugrave
Copy link
Contributor

@ugrave ugrave commented Oct 25, 2022

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Description

This PR includes pods defined via kubernetes_deployment in pod checks . PR 4 of 4.

Fixes #3690

New/Edited policies (Delete if not relevant)

  • CKV_K8S_35
    Add resource: kubernetes_deployment, kubernetes_deployment_v1
  • CKV_K8S_18
    Add resource: kubernetes_deployment, kubernetes_deployment_v1
  • CKV_K8S_17
    Add resource: kubernetes_deployment, kubernetes_deployment_v1
  • CKV_K8S_19
    Add resource: kubernetes_deployment, kubernetes_deployment_v1
  • CKV_K8S_34
    Add resource: kubernetes_deployment, kubernetes_deployment_v1

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my feature, policy, or fix is effective and works
  • New and existing tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@ugrave ugrave changed the title feat(terraform) include pods of kubernetes_deployment in kubernetes_pod checks (4/4) feat(terraform): include pods of kubernetes_deployment in kubernetes_pod checks (4/4) Oct 25, 2022
@ugrave ugrave force-pushed the deployment_pod_checks_4 branch from a526b6c to 9721fa5 Compare October 28, 2022 17:18
Copy link
Contributor

@gruebel gruebel left a comment

Choose a reason for hiding this comment

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

great job again 🏅

@gruebel gruebel merged commit 7494e1a into bridgecrewio:master Nov 7, 2022
ayajbara pushed a commit that referenced this pull request Nov 7, 2022
…pod checks (4/4) (#3738)

Include pods of kubernetes_deployment in kubernetes_pod checks (4/4)
ayajbara pushed a commit that referenced this pull request Nov 7, 2022
…pod checks (4/4) (#3738)

Include pods of kubernetes_deployment in kubernetes_pod checks (4/4)
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.

Include Pods specefied via kubernetes_deployments in kubernetes_pod checks
3 participants