Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Commit the data directory rather than creating it when the server starts #19745

Merged
merged 1 commit into from
Jan 22, 2020

Conversation

carbonin
Copy link
Member

@carbonin carbonin commented Jan 22, 2020

If we are not running a server on worker containers it feels like this
directory won't exist. I don't see why we shouldn't just have it exist
in the source tree.

Specifically I think this is important for embedded ansible in pods.

Extracted from #19734

If we are not running a server on worker containers it feels like this
directory won't exist. I don't see why we shouldn't just have it exist
in the source tree.
@miq-bot
Copy link
Member

miq-bot commented Jan 22, 2020

Checked commit carbonin@5c7dd53 with ruby 2.5.5, rubocop 0.69.0, haml-lint 0.20.0, and yamllint 1.10.0
1 file checked, 0 offenses detected
Everything looks fine. ⭐

@carbonin
Copy link
Member Author

Applies to ManageIQ/manageiq-pods#353

@Fryguy Fryguy merged commit 3229f99 into ManageIQ:master Jan 22, 2020
@Fryguy Fryguy added the core label Jan 22, 2020
@Fryguy Fryguy added this to the Sprint 129 Ending Feb 3, 2020 milestone Jan 22, 2020
@carbonin carbonin deleted the commit_the_data_directory branch January 22, 2020 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants