Fix yq expression syntax in configuration-converter#1519
Conversation
The validate_spec_components function in configuration-converter.sh was failing with "Error: bad expression, please check expression syntax" on extracting "components" keys. The original command attempted to delete non-"enabled" keys using: del(.spec.components.[] | keys[] | select(. != "enabled")) This failed because: - `.spec.components.[]` incorrectly tries to iterate over object values - `keys[]` syntax doesn't work in this pipeline context in yq v4 - `select(. != "enabled")` references values instead of key names - No proper context management for applying deletions back to document Replaced with proper yq v4 syntax using the `with()` function. Signed-off-by: Maxim Babushkin <mbabushk@redhat.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1519 +/- ##
==========================================
+ Coverage 80.94% 81.03% +0.08%
==========================================
Files 44 44
Lines 2299 2299
==========================================
+ Hits 1861 1863 +2
+ Misses 322 321 -1
+ Partials 116 115 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
e4a530f to
c743475
Compare
|
/test tools-hack-check |
|
/cherry-pick release-1.28 |
|
@MaxBab: only istio-ecosystem org members may request cherry picks. If you are already part of the org, make sure to change your membership to public. Otherwise you can still do the cherry-pick manually. DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
* upstream/main: Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
|
/cherry-pick release-1.28 |
|
/cherry-pick release-1.27 |
|
/cherry-pick release-1.26 |
|
@fjglira: new pull request created: #1524 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@fjglira: new pull request created: #1525 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
|
@fjglira: new pull request created: #1526 DetailsIn response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
* upstream/main: Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1551) docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1551) docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1551) docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Fix profile column status (istio-ecosystem#1553) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1551) docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Fix profile column status (istio-ecosystem#1553) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1551) docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Fix profile column status (istio-ecosystem#1553) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1551) docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Fix profile column status (istio-ecosystem#1553) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1551) docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1557) Fix profile column status (istio-ecosystem#1553) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1551) docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
* upstream/main: Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1557) Fix profile column status (istio-ecosystem#1553) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1551) docs: Add comprehensive Istio Ambient Mode update and waypoint proxy procedures (istio-ecosystem#1279) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1549) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1546) fixing the step numbers in our docs (istio-ecosystem#1545) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1543) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1534) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1530) update eol istio versions (istio-ecosystem#1529) Enable external registry support for OCP e2e tests in CI (istio-ecosystem#1514) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1527) Fix yq expression syntax in configuration-converter (istio-ecosystem#1519) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1520) docs: Add Multi-Primary Multi-Network Ambient mode instructions (istio-ecosystem#1505) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1516) Automator: Update dependencies in istio-ecosystem/sail-operator@main (istio-ecosystem#1515)
What type of PR is this?
What this PR does / why we need it:
The validate_spec_components function in configuration-converter.sh was failing with "Error: bad expression, please check expression syntax" on extracting "components" keys.
The original command attempted to delete non-"enabled" keys using: del(.spec.components.[] | keys[] | select(. != "enabled"))
This failed because:
.spec.components.[]incorrectly tries to iterate over object valueskeys[]syntax doesn't work in this pipeline context in yq v4select(. != "enabled")references values instead of key namesReplaced with proper yq v4 syntax using the
with()function.Which issue(s) this PR fixes:
Fixes #
Related Issue/PR #
Additional information: