-
Notifications
You must be signed in to change notification settings - Fork 104
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
Add verifier support for the special $
variable.
#1255
Conversation
This includes a test case. Also fix some typos found when reading related code.
b911edf
to
be1a5b4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice catch, this looks great.
fantastic!! I was aware we needed to handle this... happy to see a solution. I will review in detail shortly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great stuff!
This includes a test case. Also fix some typos found when reading related code. Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
) - add Toggle task for all features present in KUDO Kafka - update the minimal KUDO version to 0.11.0 because of Toggle Task usage - add a section of planned releases in `README.md` for unreleased versions to avoid any confusion - separate `deploy` and `update` plans, to avoid running pipe task in each update. This should also reduce the test duration in all CI pipelines - modify the autoTLS feature so it only generates certs when enabled - migrate mirrormaker and user workload to use `apps/v1` - remove the usage of `$root` as kudobuilder/kudo#1255 was merged and released Signed-off-by: Zain Malik <zmalikshxil@gmail.com>
What this PR does / why we need it:
This fixes the verifier for the most basic but also the most useful case in #1201 - which is to be able to refer to the "top-level" from anywhere in the tree.
It does not add support for general user-verified variables to the verifier (which I think might be much harder if not impossible for a static checker).
This includes a test case.
Also fix some typos found when reading related code.