Closed voelzmo closed 12 months ago
This PR proposes changes that would break the pipeline definition:
<invalid YAML>: while parsing a block mapping
in "<unicode string>", line 92, column 11:
vpa-recommender:
^
expected <block end>, but found '<block mapping start>'
in "<unicode string>", line 110, column 12:
vpa-admission-controller:
^
/needs second-opinion @rishabh-11 @elankath
@himanshu-kun Labels needs/@elankath, needs/@rishabh-11 do not exist.
The pipeline-definition has been fixed.
@unmarshall, @rishabh-11 You have pull request review open invite, please check
This PR proposes changes that would break the pipeline definition:
vertical-pod-autoscaler-enh_add-vpa-release-pipeline: Traceback (most recent call last):
File "/usr/lib/python3.11/site-packages/concourse/replicator.py", line 141, in render
definition_descriptor = self._render(definition_descriptor)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/site-packages/concourse/replicator.py", line 185, in _render
'definition': factory.create_pipeline_definition(),
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/site-packages/concourse/factory.py", line 83, in create_pipeline_definition
variant = self._create_variant(
^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/site-packages/concourse/factory.py", line 138, in _create_variant
variant._traits_dict = self._create_traits(raw_dict, variant_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/site-packages/concourse/factory.py", line 189, in _create_traits
traits_dict = {
^
File "/usr/lib/python3.11/site-packages/concourse/factory.py", line 190, in <dictcomp>
name: TraitsFactory.create(
^^^^^^^^^^^^^^^^^^^^^
File "/usr/lib/python3.11/site-packages/concourse/model/traits/__init__.py", line 68, in create
raise ModelValidationError('no such trait: ' + str(name))
model.base.ModelValidationError: no such trait: git_tags
The pipeline-definition has been fixed.
What this PR does / why we need it: Add a pipeline to release VPA from our fork. We're currently looking into releasing a forked VPA in the context of a complex bug where we need more debug output in the logs in case the bug occurs. This won't be a change that will be merged upstream and will only be of temporary nature.
Release note: