Skip to content

Fix Issue with nested Kubernetes Native Objects not being properly configurable in the Kubernetes AutoDiscovery Config #2982

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

Conversation

BorisShek
Copy link
Contributor

@BorisShek BorisShek commented Apr 13, 2025

Viper was misparsing the inline fields of corev1.Volume objects, causing issues with our YAML configuration files for auto-discovery. To resolve this, we are switching to sigs.k8s.io/yaml for unmarshalling. Replace time.Duration with metav1.Duration in autodiscovery_config.go structs to accommodate the way sigs.k8s.io/yaml handles string values representing time in YAML files.
Replace time.Duration with metav1.Duration in autodiscovery_config.go structs to accommodate the way sigs.k8s.io/yaml handles string values representing time in YAML files.

Description

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

Copy link

netlify bot commented Apr 13, 2025

Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit aaed2c5
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/680413ba65fa04000860bd1b

@BorisShek BorisShek moved this from Triage to To Review in secureCodeBox v4 Apr 13, 2025
@BorisShek BorisShek added bug Bugs testing Improvements or additions regarding the test setup auto-discovery dependencies Pull requests that update a dependency file and removed testing Improvements or additions regarding the test setup labels Apr 13, 2025
@BorisShek BorisShek self-assigned this Apr 15, 2025
@BorisShek BorisShek linked an issue Apr 15, 2025 that may be closed by this pull request
@BorisShek
Copy link
Contributor Author

Resolves #2724

…overy

Viper was misparsing the inline fields of corev1.Volume objects, causing issues with our YAML configuration files for auto-discovery. To resolve this, we are switching to sigs.k8s.io/yaml for unmarshalling.
Replace time.Duration with metav1.Duration in autodiscovery_config.go structs to accommodate the way sigs.k8s.io/yaml handles string values representing time in YAML files.

Signed-off-by: Boris Shek <boris.shek@iteratec.com>
…_loader.go in auto-discovery

Update auto-discovery unit test snapshot.

Signed-off-by: Boris Shek <boris.shek@iteratec.com>
Correct a minor typographical error in service_scan_controller.go regarding the RequeueAfter value.

Signed-off-by: Boris Shek <boris.shek@iteratec.com>
@BorisShek BorisShek force-pushed the scb_2724_AutoDiscovery_volume_is_propagated_as_emptyDir_to_ScheduledScan branch from 0e9d4fd to aaed2c5 Compare April 19, 2025 21:20
Copy link

@J12934 J12934 changed the title #2724 Switch from viper to sigs.k8s.io/yaml in auto-discovery Fix Issue with nested Kubernetes Native Objects not being properly configurable in the Kubernetes AutoDiscovery Config Apr 22, 2025
Copy link
Member

@J12934 J12934 left a comment

Choose a reason for hiding this comment

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

Nice 🙌

@github-project-automation github-project-automation bot moved this from To Review to Reviewer Approved in secureCodeBox v4 Apr 22, 2025
@J12934 J12934 merged commit b682db1 into secureCodeBox:main Apr 22, 2025
53 checks passed
@github-project-automation github-project-automation bot moved this from Reviewer Approved to Done in secureCodeBox v4 Apr 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-discovery bug Bugs dependencies Pull requests that update a dependency file
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

AutoDiscovery: volume is propagated as emptyDir to ScheduledScan
2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy