Skip to content
This repository was archived by the owner on Dec 7, 2022. It is now read-only.
/ devel Public archive

Starts pulp_streamer with other pulp services#15

Merged
werwty merged 1 commit intopulp:masterfrom
werwty:master
Oct 5, 2016
Merged

Starts pulp_streamer with other pulp services#15
werwty merged 1 commit intopulp:masterfrom
werwty:master

Conversation

@werwty
Copy link
Contributor

@werwty werwty commented Oct 5, 2016

This is so plugins are loaded into the streamer.

closes #2272
https://pulp.plan.io/issues/2272

@mention-bot
Copy link

@werwty, thanks for your PR! By analyzing the history of the files in this pull request, we identified @jeremycline to be a potential reviewer.

@bmbouter bmbouter self-assigned this Oct 5, 2016
Copy link
Member

@bmbouter bmbouter left a comment

Choose a reason for hiding this comment

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

Thanks @werwty

@bmbouter
Copy link
Member

bmbouter commented Oct 5, 2016

@werwty Oh one thing, the commit should probably be redone to follow the commit length guidelines

This is so plugins are loaded into the streamer.

closes #2272
https://pulp.plan.io/issues/2272
@werwty werwty changed the title Starts pulp_streamer with other pulp services so plugins are loaded i… Starts pulp_streamer with other pulp services Oct 5, 2016
Copy link
Member

@bmbouter bmbouter left a comment

Choose a reason for hiding this comment

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

Actually don't we want the streamer to be started?

@werwty
Copy link
Contributor Author

werwty commented Oct 5, 2016

streamer is started with the pstart call in vagrant-setup.sh

@werwty werwty merged commit c83dd1b into pulp:master Oct 5, 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.

3 participants