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

Empty catalog using fluxcd #6863

Closed
alvistar opened this issue Sep 22, 2023 · 2 comments · Fixed by #7219
Closed

Empty catalog using fluxcd #6863

alvistar opened this issue Sep 22, 2023 · 2 comments · Fixed by #7219
Assignees
Labels
component/plugin-flux Issue related to kubeapps plugin to manage Helm charts via Flux kind/bug An issue that reports a defect in an existing feature

Comments

@alvistar
Copy link

Describe the bug
I am using a k3s cluster with fluxcd installed. I have installed kubeapps with helm.

I can manage to see installed helm charts, but I cannot see any app in catalog

To Reproduce
Steps to reproduce the behavior:

  1. Go to catalog on top bar
  2. "The current catalog is empty."

Expected behavior
See catalog from helmrepositories resources

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop :

  • Version [e.g. 2.8.0]
  • Kubernetes version [e.g. 1.27.5]

Additional context
I see some activities on log pod:

...
{
"manager": "source-controller",
"operation": "Update",
"apiVersion": "source.toolkit.fluxcd.io/v1beta2",
"time": "2023-09-17T06:09:12Z",
"fieldsType": "FieldsV1",
"fieldsV1": {
"f:metadata": {
"f:finalizers": {
".": {},
"v:"finalizers.fluxcd.io"": {}
}
}
}
},
{
"manager": "source-controller",
"operation": "Update",
"apiVersion": "source.toolkit.fluxcd.io/v1beta2",
"time": "2023-09-21T14:49:51Z",
"fieldsType": "FieldsV1",
"fieldsV1": {
"f:status": {
"f:artifact": {
".": {},
"f:digest": {},
"f:lastUpdateTime": {},
"f:path": {},
"f:revision": {},
"f:size": {},
"f:url": {}
},
"f:conditions": {},
"f:observedGeneration": {},
"f:url": {}
}
},
"subresource": "status"
}
]
},
"spec": {
"url": "https://immich-app.github.io/immich-charts",
"interval": "5m0s",
"timeout": "1m0s",
"provider": "generic"
},
"status": {
"observedGeneration": 1,
"conditions": [
{
"type": "Ready",
"status": "True",
"observedGeneration": 1,
"lastTransitionTime": "2023-09-21T14:49:51Z",
"reason": "Succeeded",
"message": "stored artifact: revision 'sha256:d3980ddd4e3d2ee6c0b898a94c0040d937e8083dfc8024635185010da329276a'"
},
{
"type": "ArtifactInStorage",
"status": "True",
"observedGeneration": 1,
"lastTransitionTime": "2023-09-18T11:20:05Z",
"reason": "Succeeded",
"message": "stored artifact: revision 'sha256:d3980ddd4e3d2ee6c0b898a94c0040d937e8083dfc8024635185010da329276a'"
}
],
"url": "http://source-controller.flux-system.svc.cluster.local./helmrepository/apps/immich/index.yaml",
"artifact": {
"path": "helmrepository/apps/immich/index-d3980ddd4e3d2ee6c0b898a94c0040d937e8083dfc8024635185010da329276a.yaml",
"url": "http://source-controller.flux-system.svc.cluster.local./helmrepository/apps/immich/index-d3980ddd4e3d2ee6c0b898a94c0040d937e8083dfc8024635185010da329276a.yaml",
"revision": "sha256:d3980ddd4e3d2ee6c0b898a94c0040d937e8083dfc8024635185010da329276a",
"digest": "sha256:e51d5764f104cda8e7c471e5ee424f72d72d6b242b0907dd3abb0dad7d273fcb",
"lastUpdateTime": "2023-09-18T11:20:05Z",
"size": 6542
}
}
}]
I0922 07:19:39.352919 1 watcher_cache.go:613] Redis [DEL helmrepositories:apps:immich]: 0
I0922 07:19:39.352952 1 watcher_cache.go:574] -syncHandler(helmrepositories:apps:immich)
I0922 07:19:39.352991 1 watcher_cache.go:299] -processNextWorkItem()
I0922 07:19:39.353010 1 watcher_cache.go:254] +processNextWorkItem()
I0922 07:19:39.363911 1 watcher_cache.go:660] "+fetch" key="helmrepositories:apps:immich"
I0922 07:19:39.364495 1 server.go:224] -fluxv2 GetAvailablePackageSummaries

but it looks like that redis is never populated.

Other things I could look at?

@alvistar alvistar added the kind/bug An issue that reports a defect in an existing feature label Sep 22, 2023
@kvaps
Copy link
Contributor

kvaps commented Nov 26, 2023

Hi, I have the same issue with 2.9.0

@github-project-automation github-project-automation bot moved this to 🗂 Backlog in Kubeapps Nov 26, 2023
@absoludity
Copy link
Contributor

Sorry @alvistar , we missed triaging this issue at the time. Thanks @kvaps for not only updating the issue, but identifying the problem and having a go at fixing it! For the record, I've left a detailed analysis built on @kvaps's findings on the PR.

antgamdia added a commit that referenced this issue Dec 22, 2023
### Description of the change

This PR is mainly to bump up some of the dependencies used in CI.
Besides, it adds `FLUX_VERSION` to the rest of the arguments being
passed to the e2e tests.

### Benefits

Besides of keeping our deps up to date, we will be able to better
identify situations like #6863.

### Possible drawbacks

N/A

### Applicable issues

N/A 

### Additional information

Explicitly not bumping `FLUX_VERSION` to the latest here. It will be
done in a separate place.

Related to #7229

---------

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@antgamdia antgamdia added the component/plugin-flux Issue related to kubeapps plugin to manage Helm charts via Flux label Jan 8, 2024
@antgamdia antgamdia added this to the Technical debt milestone Jan 8, 2024
antgamdia added a commit that referenced this issue Jan 9, 2024
#7219)

### Description of the change

This PR is bumping up some dependencies together, as they are somehow
coupled. Besides, it performs some changes on the Flux resources
(including the changes performed in
#7132 from @kvaps; thank
you for the PR!) to make it compatible with their v2.0.0 release
onwards.

As pointed out in
#7132 (comment),
some additional changes were required, namely:

- Effectively use latest Flux version in CI, for identifying the issue.
- Replace `PollInfinite` with `PollUntilContextCancel` bc of a k8s
deprecation.
- Rename fluxcd imports to `sourcev1beta2` to clearly identify which
version we are using.
- Add TODOs for using `CertSecretRef` in the future (upcoming
deprecation, but it will be addressed in a separate PR).
- Minor change in method signature because of the kapp-ctrl upgrade.

### Benefits

We'll be able again to get dependabot's upgrades in these dependencies
and, more importantly, Kubeapps will work again with latest FluxCD
releases.

### Possible drawbacks

Conversely, Kubeapps will likely fail with Flux2 version prior to 2.0.0.
See more information and migration guide at:
https://github.com/fluxcd/flux2/releases/tag/v2.0.0


### Applicable issues

- fixes #6863

### Additional information

N/A

---------

Signed-off-by: Antonio Gamez Diaz <agamez@vmware.com>
@github-project-automation github-project-automation bot moved this from 🗂 Backlog to ✅ Done in Kubeapps Jan 9, 2024
@antgamdia antgamdia self-assigned this Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/plugin-flux Issue related to kubeapps plugin to manage Helm charts via Flux kind/bug An issue that reports a defect in an existing feature
Projects
Archived in project
4 participants