netgroup-polito / frog4-gui

Graphical User Interface for the FROG4 orchestration framework
3 stars 0 forks source link

Delete button #27

Closed ivanocerrato closed 7 years ago

ivanocerrato commented 8 years ago

Add a delete button that deletes the graph currently shown.

giacomoRanieri commented 8 years ago

This Button should delete the whole content of the graph, or just the selected element?

ivanocerrato commented 8 years ago

It would be better to introduce the possibility to delete the selected elements.. But note that, if you delete, e.g., a VNF, the command sent to the UN should also delete the flows involving such a VNF. The same applies to the endpoints

frisso commented 8 years ago

button should delete the entire graph, maybe after asking for confirmation; I work suggest some icon such http://www.softicons.com/application-icons/qetto-icons-2-by-ampeross/trash-icon

giacomoRanieri commented 8 years ago

There is no need to delete the entire graph, unless you need to delete it from the universal node. If you want to clean a graph you simply create a new one.

ivanocerrato commented 8 years ago

Why there is no need? I may want to delete a graph deployed on the UN. In fact the UN allows to delete the entire graph https://github.com/netgroup-polito/un-orchestrator/blob/master/orchestrator/README_RESTAPI.md

giacomoRanieri commented 8 years ago

This is a different matter. You have to first distinguish between a graph loaded from the universal node form a graph loaded from a different source. With these information, I've understood the problem. I'll think to a solution that could work on both repository and universal node based on the source of the graph

frisso commented 8 years ago

Agreed. Deleting a graph from the UN, without replacing it with another graph, is definitely important.

giacomoRanieri commented 7 years ago

Functionality now available on master.