Joey Armstrong | 9bb4b10 | 2024-02-10 13:09:12 -0500 | [diff] [blame^] | 1 | # Update Helm Chart for repo:votlha-openolt-adapter |
| 2 | |
| 3 | ## HOWTO: Helm Chart Update |
| 4 | |
| 5 | .. code-block:: shell-session |
| 6 | :caption: Cookbook of edits |
| 7 | |
| 8 | % git clone "ssh://gerrit.opencord.org:29418/voltha-helm-charts.git" |
| 9 | % cd voltha-helm-charts |
| 10 | % vi $(grep -r 'adapter' . | cut -d: -f1 | sort -u) |
| 11 | |
| 12 | % make lint-chart |
| 13 | % make lint-helm |
| 14 | |
| 15 | - Increment repository `VERSION file <https://gerrit.opencord.org/plugins/gitiles/voltha-openolt-adapter/+/refs/heads/master/VERSION>`_. Build, test and deploy. |
| 16 | - Modify repo:voltha-helm-charts |
| 17 | |
| 18 | - Update component version in the component chart file. |
| 19 | - Update chart version for all charts dependent on the component chart. |
| 20 | - Run ``make lint-chart lint-helm`` to validate chart changes. |
| 21 | - chart:voltha, chart:voltha-infra and chart:voltha-stack can be confusing |
| 22 | due to a layer of indirection: |
| 23 | |
| 24 | - chart:voltha-{infra,stack} are not not named for code repositories, |
| 25 | they are a container chart used to capture a list of dependencies |
| 26 | and versions. |
| 27 | - chart:voltha depends on these two charts which in turn depend on |
| 28 | sets of voltha components & versions. |
| 29 | - When ``make lint-chart`` report this error |
| 30 | |
| 31 | - [ERROR] Chart modified but version unchanged: voltha/Chart.yaml (2.12.11) |
| 32 | - It will imply that a component like openolt-adapter has been updated |
| 33 | but chart:voltha and chart:voltha-{infra,stack} have not. |
| 34 | |
| 35 | .. list-table:: repo:voltha-helm-charts edits |
| 36 | :widths: 10, 20, 60 |
| 37 | :header-rows: 1 |
| 38 | |
| 39 | * - Repository Path |
| 40 | - Key |
| 41 | - Description |
| 42 | * - `voltha-adapter-openolt/Chart.yaml <https://gerrit.opencord.org/plugins/gitiles/voltha-helm-charts/+/refs/heads/master/voltha-adapter-openolt/Chart.yaml>`_ |
| 43 | - Chart.yaml : appVersion |
| 44 | - Assign `repo:voltha-openolt-adapter::VERSION <https://gerrit.opencord.org/plugins/gitiles/voltha-openolt-adapter/+/refs/heads/master/VERSION>`_ to `Chart.yaml::appVersion <https://gerrit.opencord.org/plugins/gitiles/voltha-helm-charts/+/refs/heads/master/voltha-adapter-openolt/Chart.yaml#35>`_ |
| 45 | * - voltha-infra/Chart.yaml |
| 46 | - |
| 47 | - |
| 48 | * - voltha-stack/Chart.yaml |
| 49 | - dependencies : name=voltha-adapter-openolt : version |
| 50 | - Component openolt-adapter modified so update `dependencies : name='voltha-adapter-olt' : version="2.12.6" <https://gerrit.opencord.org/plugins/gitiles/voltha-helm-charts/+/refs/heads/master/voltha-stack/Chart.yaml#38>`_ |
| 51 | * - voltha-stack/Chart.yaml |
| 52 | - Chart.yaml : version |
| 53 | - voltha-adapter-openolt/Chart.yaml modified so notify voltha-stack. Update `version="2.12.6" <https://gerrit.opencord.org/plugins/gitiles/voltha-helm-charts/+/refs/heads/master/voltha-stack/Chart.yaml#30>`_. |
| 54 | * - voltha-stack/Chart.yaml |
| 55 | - dependency : name=voltha : version |
| 56 | - voltha-stack/Chart.yaml modified so notify chart voltha. Update `dependencies : name=voltha : version="2.12.2" <https://gerrit.opencord.org/plugins/gitiles/voltha-helm-charts/+/refs/heads/master/voltha-stack/Chart.yaml#34>`_ |
| 57 | * - voltha/Chart.yaml |
| 58 | - Chart.yaml : version |
| 59 | - voltha/Chart.yaml updated so modify chart `Chart.yaml : version='"2.12.11"' <https://gerrit.opencord.org/plugins/gitiles/voltha-helm-charts/+/refs/heads/master/voltha/Chart.yaml#17>`_ |
| 60 | * - voltha/Chart.yaml |
| 61 | - Chart.yaml : version |
| 62 | - Update `voltha/Chart.yaml : version='"2.12.11"' <https://gerrit.opencord.org/plugins/gitiles/voltha-helm-charts/+/refs/heads/master/voltha/Chart.yaml#17>`_ to match `voltha-stack:dependencies : name=voltha : version='"2.12.11"' <https://gerrit.opencord.org/plugins/gitiles/voltha-helm-charts/+/refs/heads/master/voltha/Chart.yaml#17>`_. |
| 63 | |
| 64 | ### Extra Credit |
| 65 | |
| 66 | - Update copyright notices :ref:`Release Task Copyright` |
| 67 | - make lint # lint everything |
| 68 | |
| 69 | .. seealso:: |
| 70 | |
| 71 | `voltha-docs::charts <../charts/index>` |