#4: Change machine_id, phys_id, netdev_id from int to str -------------------------+------------------------- Reporter: rpazdera | Owner: rpazdera@… Type: enhancement | Status: new Priority: minor | Milestone: Component: component1 | Version: Keywords: | Blocked By: Blocking: | -------------------------+------------------------- It would be nice to be able to have a possibility of giving symbolic names to either machines, physical devices or configured interfaces.
Currently only integer values are accepted as machine_id, netdev_phys_id and interface_id.
#4: Change machine_id, phys_id, netdev_id from int to str --------------------------+------------------------ Reporter: rpazdera | Owner: rpazdera@… Type: enhancement | Status: new Priority: minor | Milestone: Component: component1 | Version: Resolution: | Keywords: Blocked By: | Blocking: --------------------------+------------------------ Changes (by jirka):
* cc: lnst-developers@… (removed)
#4: Change machine_id, phys_id, netdev_id from int to str --------------------------+---------------------- Reporter: rpazdera | Owner: olichtne Type: enhancement | Status: new Priority: minor | Milestone: Component: component1 | Version: Resolution: | Keywords: Blocked By: | Blocking: --------------------------+---------------------- Changes (by rpazdera):
* owner: rpazdera@… => olichtne
#4: Change machine_id, phys_id, netdev_id from int to str --------------------------+---------------------- Reporter: rpazdera | Owner: olichtne Type: enhancement | Status: assigned Priority: minor | Milestone: Component: component1 | Version: Resolution: | Keywords: Blocked By: | Blocking: --------------------------+---------------------- Changes (by olichtne):
* status: new => assigned
#4: Change machine_id, phys_id, netdev_id from int to str --------------------------+---------------------- Reporter: rpazdera | Owner: olichtne Type: enhancement | Status: assigned Priority: minor | Milestone: Component: component1 | Version: Resolution: | Keywords: Blocked By: | Blocking: --------------------------+----------------------
Comment (by olichtne):
While I was looking into the code to find out how much needs to be changed I noticed that bg_id, used for identification of background processes, can be changed to string as well. That way you could give the command a more comprehensive id for example "background_file_transfer" (or something along those lines) the same id would be then used in the corresponding wait/interrupt/kill command. This could make the connection between these two commands a bit more visible.[[BR]] Should I make this change as well?
#4: Change machine_id, phys_id, netdev_id from int to str --------------------------+---------------------- Reporter: rpazdera | Owner: olichtne Type: enhancement | Status: assigned Priority: minor | Milestone: Component: component1 | Version: Resolution: | Keywords: Blocked By: | Blocking: --------------------------+----------------------
Comment (by jtluka):
Replying to [comment:4 olichtne]:
While I was looking into the code to find out how much needs to be
changed I noticed that bg_id, used for identification of background processes, can be changed to string as well. That way you could give the command a more comprehensive id for example "background_file_transfer" (or something along those lines) the same id would be then used in the corresponding wait/interrupt/kill command. This could make the connection between these two commands a bit more visible.[[BR]]
Should I make this change as well?
Sounds good to me. This would enhance readability of the recipe.
#4: Change machine_id, phys_id, netdev_id from int to str --------------------------+---------------------- Reporter: rpazdera | Owner: olichtne Type: enhancement | Status: assigned Priority: minor | Milestone: Component: component1 | Version: Resolution: | Keywords: Blocked By: | Blocking: --------------------------+----------------------
Comment (by olichtne):
Patches sent:[[BR]] https://lists.fedorahosted.org/pipermail/lnst- developers/2012-August/000399.html [[BR]] https://lists.fedorahosted.org/pipermail/lnst- developers/2012-August/000400.html [[BR]] https://lists.fedorahosted.org/pipermail/lnst- developers/2012-August/000401.html [[BR]] https://lists.fedorahosted.org/pipermail/lnst- developers/2012-August/000402.html
#4: Change machine_id, phys_id, netdev_id from int to str --------------------------+---------------------- Reporter: rpazdera | Owner: olichtne Type: enhancement | Status: closed Priority: minor | Milestone: Component: component1 | Version: Resolution: fixed | Keywords: Blocked By: | Blocking: --------------------------+---------------------- Changes (by olichtne):
* status: assigned => closed * resolution: => fixed
lnst-developers@lists.fedorahosted.org