Skip to content
This repository has been archived by the owner on Jan 5, 2023. It is now read-only.

GS 2 [WIP] #4

Closed
wants to merge 18 commits into from
Closed

GS 2 [WIP] #4

wants to merge 18 commits into from

Conversation

zeisss
Copy link

@zeisss zeisss commented Aug 26, 2015

No description provided.

retChan := make(chan string)
m.stopUnit(name, retChan)

jobReply := <-retChan
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we want to time this and a log.Info beforehand?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nop. its a queue_append operation, the time is meaningless.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then where is the block happening? I would assume that stopUnit() is quick because it does stuff in a goroutine / async and this channel is written to, when the stop finished. So the read here is blocking / slow. So time should be meaning full.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

jobReply := <-retChan

the block happens here. this call only returns after systemd has the stop job enqueued. a stop job being enqueued means that systemd will stop it, it doesnt block until it is stopped.
That feature is part of something bigger and unrelated.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

k

@xh3b4sd
Copy link

xh3b4sd commented Sep 4, 2015

Is this the version we currently use? I have the feeling we miss some commits here.

@htr
Copy link

htr commented Sep 5, 2015

this is the version we are using and there are no commits missing.

@hectorj2f
Copy link

ping @zeisss Should we close this PR ?

@zeisss
Copy link
Author

zeisss commented Apr 18, 2016

@hectorj2f I created this PR as a proxy for @htr as he forgot it. I have no insight into the actual content, sorry. I cannot judge this.

@htr htr closed this Apr 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants