Project

General

Profile

Bug #14519

deleting a container using hammer silently fails without error

Added by Daniel Lobato Garcia almost 4 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
-
Difficulty:
Triaged:
No
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1324211
Description of problem:
deleting a container in hammer is successful but the container is not actually removed from external docker compute resource

Version-Release number of selected component (if applicable):
Satellite-6.2-beta-snap6.1

How reproducible:
Always

Steps to Reproduce:
1. Create an external docker compute resource
2. Create a docker container using the external docker computer resource
3. delete the docker container using hammer
  1. hammer u admin -p changeme docker container list
    ---|--------------|----------------------------------------------------------------|--------|---------|----------------

    ID | NAME | IMAGE REPOSITORY | TAG | COMMAND | COMPUTE RESOURCE
    ---|--------------|----------------------------------------------------------------|--------|---------|-----------------
    5 | container_02 | default_organization-dev1-docker_cv-dockerproduct1-dockerrepo1 | latest | top | docker_cr
    ---|--------------|----------------------------------------------------------------|--------|---------|-----------------
  1. hammer -u admin -p changeme docker container delete --name container_02
    Docker container deleted
  1. hammer u admin -p changeme docker container list
    ---|------|------------------|-----|---------|----------------

    ID | NAME | IMAGE REPOSITORY | TAG | COMMAND | COMPUTE RESOURCE
    ---|------|------------------|-----|---------|-----------------
Actual results:
Hammer delete was successful as shown above. But
  • Container is still shown as running in Satellite UI
  • Container is still running in the external docker compute resource

Expected results:
Container should be removed from Satellite UI too.
Container should be removed in external docker compute resource too.

Additional info:


Related issues

Has duplicate Docker - Bug #14039: Containers can't be deleted via API or hammerDuplicate2016-03-03

Associated revisions

Revision bbb77219 (diff)
Added by Daniel Lobato Garcia almost 4 years ago

Fixes #14519 - Removing containers from API removes them from Docker

Before this fix, the API was just removing the container in the Foreman
database. This commit includes more tests for both the UI and API
container deletion to ensure this area is a bit more solid.

Revision bbb77219 (diff)
Added by Daniel Lobato Garcia almost 4 years ago

Fixes #14519 - Removing containers from API removes them from Docker

Before this fix, the API was just removing the container in the Foreman
database. This commit includes more tests for both the UI and API
container deletion to ensure this area is a bit more solid.

History

#1 Updated by The Foreman Bot almost 4 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman-docker/pull/154 added

#2 Updated by Daniel Lobato Garcia almost 4 years ago

  • Has duplicate Bug #14039: Containers can't be deleted via API or hammer added

#3 Updated by Daniel Lobato Garcia almost 4 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Also available in: Atom PDF