renaming id() to state clearly that it's related to the networkAttachement container target
Signed-off-by: Yassine TIJANI <yasstij11@gmail.com> Upstream-commit: 27c0131a43b1f4956446467273e82826264d6c35 Component: engine
This commit is contained in:
@@ -199,7 +199,7 @@ func (c *containerAdapter) networkAttach(ctx context.Context) error {
|
||||
}
|
||||
}
|
||||
|
||||
return c.backend.UpdateAttachment(networkName, networkID, c.container.id(), config)
|
||||
return c.backend.UpdateAttachment(networkName, networkID, c.container.networkAttachmentContainerID(), config)
|
||||
}
|
||||
|
||||
func (c *containerAdapter) waitForDetach(ctx context.Context) error {
|
||||
@@ -218,7 +218,7 @@ func (c *containerAdapter) waitForDetach(ctx context.Context) error {
|
||||
}
|
||||
}
|
||||
|
||||
return c.backend.WaitForDetachment(ctx, networkName, networkID, c.container.taskID(), c.container.id())
|
||||
return c.backend.WaitForDetachment(ctx, networkName, networkID, c.container.taskID(), c.container.networkAttachmentContainerID())
|
||||
}
|
||||
|
||||
func (c *containerAdapter) create(ctx context.Context) error {
|
||||
|
||||
@@ -89,7 +89,7 @@ func (c *containerConfig) setTask(t *api.Task) error {
|
||||
return nil
|
||||
}
|
||||
|
||||
func (c *containerConfig) id() string {
|
||||
func (c *containerConfig) networkAttachmentContainerID() string {
|
||||
attachment := c.task.Spec.GetAttachment()
|
||||
if attachment == nil {
|
||||
return ""
|
||||
@@ -115,7 +115,7 @@ func (c *containerConfig) nameOrID() string {
|
||||
return c.name()
|
||||
}
|
||||
|
||||
return c.id()
|
||||
return c.networkAttachmentContainerID()
|
||||
}
|
||||
|
||||
func (c *containerConfig) name() string {
|
||||
|
||||
Reference in New Issue
Block a user