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

Add setuptools and rpm setup capabilities #21

Merged
merged 20 commits into from Mar 16, 2016

Conversation

sampathP
Copy link
Contributor

@sampathP sampathP commented Mar 9, 2016

This patch add setuptools and rpm setup capabilities to masakari

Implement setuptools with pbr packaging feature for,
masakari-controller and masakari-instancemonitor.

rpm spec files for each project and located under,
[project_name]/rpmbuild/SPECS/*.spec
systemd scripts are under,
[project_name]/etc/systemd/system/[project_name].service

setuptools with pbr used for python packages such as
masakari-controller and masakari-instancemoniror.
For, shell scripts in masakari-processmonitor and
masakari-hostmonitor uses spec file to build rpms.
logger only create the log file but not the path.
Need to create the path if not exist.
Add spec files to each masakari-* to build rpms.
Add systemd scripts to control the services.
Add howtobuild_rpms files.
TODO: fill them out
Fix spec file for masakari-instancemonitor
How to setup masakari-controller
How to build rpms from source
Add python-novaclient to requirements
@sampathP
Copy link
Contributor Author

sampathP commented Mar 9, 2016

To test this code,
Pull request to send to masakari-deploy for related deploy patch.
Or use following repo,
Repo: https://github.com/sampathP/masakari-deploy/
Branch: feature/centos-support
which is same as the above pull req.

@muroi
Copy link
Contributor

muroi commented Mar 10, 2016

Overall, LGTM. If possible, could you surround commands in README with sh or something like that to easy to read in github page.

@sampathP
Copy link
Contributor Author

@muroi Thank you for the review.
markdown the README is done.

@toshikazu0314
Copy link
Contributor

looks good to me. Just need to resolve conflicts.

@sampathP
Copy link
Contributor Author

Conflicts resolved

@sampathP
Copy link
Contributor Author

#18 and this PR add .gitignore file but file content is slightly different.
I could rebase, but it will flush all the comment messages on this PR.
So, I will push a another patch for .gitignore after merge this.

muroi added a commit that referenced this pull request Mar 16, 2016
Add setuptools and rpm setup capabilities
Close #20
@muroi muroi merged commit 14bbace into ntt-sic:master Mar 16, 2016
@sampathP sampathP deleted the pkging/setupwithpbr branch July 1, 2016 05:05
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.

None yet

3 participants