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

Submodules aren't cloned recursively anymore since Fleet 0.9.x #2492

Closed
1 task done
puffitos opened this issue Jun 6, 2024 · 2 comments
Closed
1 task done

Submodules aren't cloned recursively anymore since Fleet 0.9.x #2492

puffitos opened this issue Jun 6, 2024 · 2 comments
Assignees
Labels
Milestone

Comments

@puffitos
Copy link
Contributor

puffitos commented Jun 6, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Current Behavior

After updating to Rancher 2.8.x (and thus getting Fleet 0.9.x), we noticed that some gitrepos which dependend on submodules being recursively cloned as well don't get updated anymore. The Gitjob pods fail with an error in the lines of "failed to download ".

This happens apparently, because the new gitjob gitcloner command that took the place of the tekton containers doesn't clone recursively:

https://github.com/rancher/gitjob/blob/1495df9fee27de43ec8104faece17028a96c7459/cmd/gitcloner/gogit/cloner.go#L71C1-L83C1

This results in our bundles being stuck and having to apply some workaround (like moving our charts/manifests from the local folder to a remote location - which is OK for the time being).

Expected Behavior

The functionality of recursively cloning submodules is still present in the current fleet version.

Steps To Reproduce

  1. Install Fleet 0.8.x (before gitjob became the default cloner for gitrepos)
  2. Deploy a Gitrepo that targets a repository that uses submodules, which include a helm chart (for example, just add a the https://github.com/bitnami/charts as a submodule and instruct fleet to deploy the chart: ./charts/bitnami/nginx/ chart)
  3. See the pod get deployed
  4. Update to fleet 0.9.x
  5. Fleet gitjob for the GitRepo should fail after a force update, because submodule isn't pulled and cannot be found

Environment

- Architecture: amd64
- Fleet Version: 0.9.2
- Cluster: 
  - Provider: seems irrelevant
  - Options: seems irrelevant
  - Kubernetes Version: 1.27.13

Logs

No response

Anything else?

No response

@rancherbot rancherbot added this to Fleet Jun 6, 2024
@github-project-automation github-project-automation bot moved this to 🆕 New in Fleet Jun 6, 2024
@manno manno moved this from 🆕 New to To Triage in Fleet Jun 26, 2024
@manno manno added this to the v2.9.0 milestone Jun 26, 2024
@manno manno moved this from To Triage to 👀 In review in Fleet Jun 26, 2024
@manno manno moved this from 👀 In review to Needs QA review in Fleet Jul 3, 2024
@sbulage
Copy link
Contributor

sbulage commented Jul 9, 2024

QA Template followed: #2557 (comment)

@sbulage sbulage self-assigned this Jul 9, 2024
@sbulage
Copy link
Contributor

sbulage commented Jul 9, 2024

System Information Before Upgrade After Upgrade
Rancher Version 2.8.5 2.9.0-alpha7
Fleet Version 0.9.5 0.10.0-rc.18

Steps Performed:

Before upgrade.

  1. Created GitRepo using @manno's fleet-experiments with path: fleet-test-data/qa-test-apps/nginx-app.
  2. Wait for the resources to be deployed.
  3. There is error in the job pod, stating "No resource found at the following paths to deploy: [fleet-test-data/qa-test-apps/nginx-app]"
  4. Job pods are continuously created and trying to load the submodule but failing at the end.

After Upgrade.

  1. Upgraded cluster to 2.9.0-alpha7 which include fix.
  2. Wait for the Rancher and Fleet to install in the cluster.
  3. Waited for Gitjob/Job pod to be created.
  4. The GitRepo still showing same status i.e. Error.
  5. Applied Force Update on GitRepo and wait for it's status to change.
  6. Verified that GitRepo is showing Active state and respective applications are installed via git submodule path.

Screenshots Before and After installation of Rancher.
Before Upgrade

Screenshot from 2024-07-08 20-21-35


After Upgrade

Screenshot from 2024-07-08 22-48-10

@sbulage sbulage closed this as completed Jul 9, 2024
@github-project-automation github-project-automation bot moved this from Needs QA review to ✅ Done in Fleet Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

3 participants