* Add E2E test for external network sources

* Addressed Gavin's comments

* Added feature flag protection

* Fix style

* Broke down the tests a bit

* Addressed Viswa's commenta

* Increased check no edge timeout

* Changed verifyNoEdge to wait first

* Skipped tc #4 for now

* Fix style

(cherry picked from commit 4f18d0c)

# This is the 1st commit message:

WIP: Helm changes for config-controller

# The commit message #2 will be skipped:

# WIP: Helm changes for config-controller

# The commit message #3 will be skipped:

# helm ignore

# The commit message #4 will be skipped:

# remove comment block from deployment yaml

# The commit message #5 will be skipped:

# manager

# The commit message #6 will be skipped:

# remove indent

# The commit message #7 will be skipped:

# more fixes to indentation

…r CLI

- Add checked type assertions in echoReplacedVersion
- Fix array bounds check (>= 3 not >= 2)
- Add validation for --related-images-mode flag
- Replace custom string functions with strings package
- Unify skip version parsing logic with patch.go

User request: Fix the 3 critical issues and Important issue #4 (use standard library) identified in the code quality review for Task 7.

This change addresses all critical safety issues:
1. Type assertions now properly check success and return descriptive errors
2. Array bounds bug fixed in version parsing (no longer needed with unified logic)
3. Related-images-mode flag validates against allowed values: downstream, omit, konflux
4. Custom string manipulation replaced with standard strings.Split, strings.TrimSpace, strings.TrimPrefix
5. Version parsing unified with patch.go approach using strings.TrimPrefix

All existing tests pass.

Note: Code partially generated with AI assistance.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

…r CLI

- Add checked type assertions in echoReplacedVersion
- Fix array bounds check (>= 3 not >= 2)
- Add validation for --related-images-mode flag
- Replace custom string functions with strings package
- Unify skip version parsing logic with patch.go

User request: Fix the 3 critical issues and Important issue #4 (use standard library) identified in the code quality review for Task 7.

This change addresses all critical safety issues:
1. Type assertions now properly check success and return descriptive errors
2. Array bounds bug fixed in version parsing (no longer needed with unified logic)
3. Related-images-mode flag validates against allowed values: downstream, omit, konflux
4. Custom string manipulation replaced with standard strings.Split, strings.TrimSpace, strings.TrimPrefix
5. Version parsing unified with patch.go approach using strings.TrimPrefix

All existing tests pass.

Note: Code partially generated with AI assistance.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

…r CLI

- Add checked type assertions in echoReplacedVersion
- Fix array bounds check (>= 3 not >= 2)
- Add validation for --related-images-mode flag
- Replace custom string functions with strings package
- Unify skip version parsing logic with patch.go

User request: Fix the 3 critical issues and Important issue #4 (use standard library) identified in the code quality review for Task 7.

This change addresses all critical safety issues:
1. Type assertions now properly check success and return descriptive errors
2. Array bounds bug fixed in version parsing (no longer needed with unified logic)
3. Related-images-mode flag validates against allowed values: downstream, omit, konflux
4. Custom string manipulation replaced with standard strings.Split, strings.TrimSpace, strings.TrimPrefix
5. Version parsing unified with patch.go approach using strings.TrimPrefix

All existing tests pass.

Note: Code partially generated with AI assistance.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Add a dedicated `cve` column with btree index to ImageCVEInfo table to
enable efficient lookups by CVE name without parsing the composite ID.

This allows queries like:
  SELECT min(first_system_occurrence) FROM image_cve_infos WHERE cve=$1;

Previously, the CVE was only stored as part of the composite primary key
(cve#package#datasource), requiring string parsing for CVE lookups.

Changes:
- Add cve field (field #4) to ImageCVEInfo proto with btree index
- Populate Cve field in both image datastore implementations
- Update tests to include the new field
- Regenerate postgres schema and proto bindings

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

…r CLI

- Add checked type assertions in echoReplacedVersion
- Fix array bounds check (>= 3 not >= 2)
- Add validation for --related-images-mode flag
- Replace custom string functions with strings package
- Unify skip version parsing logic with patch.go

User request: Fix the 3 critical issues and Important issue #4 (use standard library) identified in the code quality review for Task 7.

This change addresses all critical safety issues:
1. Type assertions now properly check success and return descriptive errors
2. Array bounds bug fixed in version parsing (no longer needed with unified logic)
3. Related-images-mode flag validates against allowed values: downstream, omit, konflux
4. Custom string manipulation replaced with standard strings.Split, strings.TrimSpace, strings.TrimPrefix
5. Version parsing unified with patch.go approach using strings.TrimPrefix

All existing tests pass.

Note: Code partially generated with AI assistance.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

…r CLI

- Add checked type assertions in echoReplacedVersion
- Fix array bounds check (>= 3 not >= 2)
- Add validation for --related-images-mode flag
- Replace custom string functions with strings package
- Unify skip version parsing logic with patch.go

User request: Fix the 3 critical issues and Important issue #4 (use standard library) identified in the code quality review for Task 7.

This change addresses all critical safety issues:
1. Type assertions now properly check success and return descriptive errors
2. Array bounds bug fixed in version parsing (no longer needed with unified logic)
3. Related-images-mode flag validates against allowed values: downstream, omit, konflux
4. Custom string manipulation replaced with standard strings.Split, strings.TrimSpace, strings.TrimPrefix
5. Version parsing unified with patch.go approach using strings.TrimPrefix

All existing tests pass.

Note: Code partially generated with AI assistance.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>