Skip to content
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

App version not reported anymore for Helm chart (since 0.1.26) #694

Closed
Constantin07 opened this issue Jun 21, 2023 · 4 comments
Closed

App version not reported anymore for Helm chart (since 0.1.26) #694

Constantin07 opened this issue Jun 21, 2023 · 4 comments

Comments

@Constantin07
Copy link

Describe the question/issue

Looks like App version is not reported anymore in the latest Helm chart version - just stable shown.
What version exactly the stable refers to ?

The stable is not SemVer compliant and to be honest is confusing as it's not clear what version from GitHub release is going to install.

Configuration

Just trying to update to the latest using Helm charts.

Fluent Bit Log Output

N/A

Fluent Bit Version Info

Helm chart version 0.1.26

Cluster Details

N/A

Application Details

N/A

Steps to reproduce issue

helm repo add eks https://aws.github.io/eks-charts
$ helm repo update
$ helm search repo eks/aws-for-fluent-bit -l
NAME                  	CHART VERSION	APP VERSION	DESCRIPTION
eks/aws-for-fluent-bit	0.1.26       	stable     	A Helm chart to deploy aws-for-fluent-bit project
eks/aws-for-fluent-bit	0.1.25       	2.28.4     	A Helm chart to deploy aws-for-fluent-bit project
eks/aws-for-fluent-bit	0.1.24       	2.28.4     	A Helm chart to deploy aws-for-fluent-bit project
eks/aws-for-fluent-bit	0.1.23       	2.21.5     	A Helm chart to deploy aws-for-fluent-bit project
...
@PettitWesley
Copy link
Contributor

PettitWesley added a commit to PettitWesley/eks-charts that referenced this issue Jun 22, 2023
PettitWesley added a commit to PettitWesley/eks-charts that referenced this issue Jun 23, 2023
@Constantin07 Constantin07 changed the title App version hot reported anymore for Helm chart (since 0.1.26) App version not reported anymore for Helm chart (since 0.1.26) Jun 23, 2023
PettitWesley added a commit to aws/eks-charts that referenced this issue Jun 23, 2023
@PettitWesley
Copy link
Contributor

New chart version: https://github.com/aws/eks-charts/releases/tag/v0.0.140

@PettitWesley
Copy link
Contributor

I've added a note to try to remember to upgrade the chart image tag every time we upgrade stable. Resolving this issue.

@Constantin07
Copy link
Author

Constantin07 commented Jun 25, 2023

Much appreciated @PettitWesley.

I confirm it does report the version properly now:

$ helm search repo aws-for-fluent-bit
NAME                  	CHART VERSION	APP VERSION	DESCRIPTION
eks/aws-for-fluent-bit	0.1.27       	2.31.11    	A Helm chart to deploy aws-for-fluent-bit project

changhyuni pushed a commit to changhyuni/eks-charts that referenced this issue Jul 13, 2023
PettitWesley added a commit to aws/eks-charts that referenced this issue Jul 26, 2023
…lugin (#951)

* Add template high performance kinesis stream plugin

* update readme.md

* update readme kinesis streams url

* update reademe

* fix typo

* rename readme.md url name

* fix typo

* chart updates for VPC CNI v1.13.0 release (#952)

* appmesh-controller: v1.12.1 (#954)

Co-authored-by: eks-bot <[email protected]>

* fix(fluentbit): rewrite erroneous ident

* fix(fluentbit): rewrite erroneous ident

* Bumping aws-for-fluent-bit helm chart version
- 2.28.4 --> stable (less manual updates)
- version 0.1.25 --> 0.1.26
- update readme

* VPC CNI updates for v1.13.2 release (#962)

* aws-load-balancer-controller: v2.5.3 (#968)

Co-authored-by: eks-bot <[email protected]>

* aws-for-fluent-bit: switch to hardcode stable version to address user complaints aws/aws-for-fluent-bit#694

Signed-off-by: Wesley Pettit <[email protected]>

* fix document

* fix chart version

---------

Signed-off-by: Wesley Pettit <[email protected]>
Co-authored-by: june.i <[email protected]>
Co-authored-by: Jeffrey Nelson <[email protected]>
Co-authored-by: Yuanshun Dong <[email protected]>
Co-authored-by: eks-bot <[email protected]>
Co-authored-by: Julien Boulanger <[email protected]>
Co-authored-by: Shelby Hagman <[email protected]>
Co-authored-by: Olivia Song <[email protected]>
Co-authored-by: Wesley Pettit <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants
@Constantin07 @PettitWesley and others