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

Changing target and subinstallation names #796

Open
robertgraeff opened this issue Jul 27, 2023 · 2 comments
Open

Changing target and subinstallation names #796

robertgraeff opened this issue Jul 27, 2023 · 2 comments
Labels
area/robustness Robustness, reliability, resilience related kind/bug Bug lifecycle/stale Nobody worked on this for 6 months (will further age) priority/3 Priority (lower number equals higher priority)

Comments

@robertgraeff
Copy link
Contributor

robertgraeff commented Jul 27, 2023

How to categorize this issue?

/area robustness
/kind bug
/priority 3

What happened:

I have successfully deployed an installation with three subinstallations. (One subinstallation would probably suffice to reproduce the error.) Each of the three subinstallations has one deployitem.

In a new version, I change the following in the blueprint of the root installation:

  • the name of the target import parameter,
  • the names of the three subinstallations.

I update the installation accordingly. After the update, the objects remains in the following state:

▶ landscaper-cli inst inspect -n example scaling-many-deployitems -e
[CleanupOrphaned] Installation scaling-many-deployitems
    Last error: some orphaned subinstallations are still deleting
    ├── [🏗️ Deleting] Installation subinst-scaling-item000-phfbw
    │   Last error: deletion of some sub objects pending
    │   └── [🏗️ Deleting] Execution subinst-scaling-item000-phfbw
    │       └── [✅ Succeeded] DeployItem subinst-scaling-item000-phfbw-default-deploy-item-tz66h
    ├── [🏗️ Deleting] Installation subinst-scaling-item001-n94q4
    │   Last error: deletion of some sub objects pending
    │   └── [🏗️ Deleting] Execution subinst-scaling-item001-n94q4
    │       └── [✅ Succeeded] DeployItem subinst-scaling-item001-n94q4-default-deploy-item-k74l2
    ├── [🏗️ Deleting] Installation subinst-scaling-item002-xh2mc
    │   Last error: deletion of some sub objects pending
    │   └── [🏗️ Deleting] Execution subinst-scaling-item002-xh2mc
    │       └── [✅ Succeeded] DeployItem subinst-scaling-item002-xh2mc-default-deploy-item-htgbh
    ├── [] Installation subinst2-scaling-item000-l65sd
    ├── [] Installation subinst2-scaling-item001-zdz52
    └── [] Installation subinst2-scaling-item002-rq59g

Three new subinstallation subinst2-... have been created. The three old (orphaned) subinstallations are about to be deleted. They have a new jobID. However, no deployer picks them up, so that their phase is still Succeeded from before.

After 5 minutes, all objects fail, because the three old (orphaned) deployitems have not been picked up by a deployer. They are not being picked up, because they still reference the old target which no longer exists. (The original target is unchanged. But its copy in the context of the root installation has a new name. The name of the target import parameter in the blueprint has changed, and the name of the copied target is derived from it.)

A delete will only delete the three new subinstallations. After that, also a landscaper-cli inst force-delete does not work, because the object tree is no longer complete.

What you expected to happen:

How to reproduce it (as minimally and precisely as possible):

Anything else we need to know?:

Environment:

  • Landscaper version:
  • Kubernetes version (use kubectl version):
  • Others:
@gardener-robot gardener-robot added the priority/3 Priority (lower number equals higher priority) label Jul 27, 2023
@gardener-robot
Copy link

@robertgraeff Label area/todo does not exist.

@robertgraeff robertgraeff added the area/robustness Robustness, reliability, resilience related label Jul 27, 2023
@achimweigel
Copy link
Contributor

@robertgraeff shouldn't we move this to the internal project and put it on the todo list?

@gardener-robot gardener-robot added the lifecycle/stale Nobody worked on this for 6 months (will further age) label Apr 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/robustness Robustness, reliability, resilience related kind/bug Bug lifecycle/stale Nobody worked on this for 6 months (will further age) priority/3 Priority (lower number equals higher priority)
Projects
None yet
Development

No branches or pull requests

3 participants