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

Allow to build uchiwa packages in docker #718

Merged
merged 1 commit into from
Jan 10, 2018

Conversation

SergK
Copy link
Contributor

@SergK SergK commented Sep 28, 2017

It's really helpful to have an ability to build uchiwa package
in docker image, without installing additional dependencies on
host system

Description

Allow to build uchiwa package in docker

Related Issue

Motivation and Context

We need to have ability for simple uchiwa packaging

How Has This Been Tested?

No changes in code itself, just additional feature with two docker commands:

  • build docker image
  • build package with docker image

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • [+] New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • [+] My change requires a change to the documentation.
  • [+] I have updated the documentation accordingly.
  • [+] I have read the CONTRIBUTING document.
  • [+] I have added tests to cover my changes.
  • [+] All new and existing tests passed.

@codecov-io
Copy link

codecov-io commented Sep 28, 2017

Codecov Report

Merging #718 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #718   +/-   ##
=======================================
  Coverage   22.39%   22.39%           
=======================================
  Files          28       28           
  Lines        2353     2353           
=======================================
  Hits          527      527           
  Misses       1766     1766           
  Partials       60       60

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 33a6263...6e4251d. Read the comment docs.

@SergK SergK force-pushed the build_packages_in_docker branch from 6ac8a07 to 2f7a516 Compare September 29, 2017 06:53
@SergK
Copy link
Contributor Author

SergK commented Sep 29, 2017

@palourde hi, can you please review PR? Thank you

Copy link

@yakirgb yakirgb left a comment

Choose a reason for hiding this comment

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

About readme file, maybe Uchiwa Docs is better place to examples.

@SergK SergK force-pushed the build_packages_in_docker branch from 2f7a516 to fcab61f Compare October 2, 2017 21:22
@palourde
Copy link
Contributor

@SergK As mentioned, could you remove the documentation in the README and open a PR against https://github.com/palourde/uchiwa-docs instead?

Thanks!

@SergK SergK force-pushed the build_packages_in_docker branch from fcab61f to 1463ae4 Compare December 15, 2017 20:13
@SergK
Copy link
Contributor Author

SergK commented Dec 15, 2017

@palourde rebased on latest master and updated docs palourde/uchiwa-docs#14

It's really helpful to have an ability to build uchiwa package
in docker image, without installing additional dependencies on
host system
@SergK SergK force-pushed the build_packages_in_docker branch from 1463ae4 to 6e4251d Compare December 17, 2017 20:04
@palourde palourde added this to the 1.1.1 milestone Jan 10, 2018
@palourde palourde merged commit f39066f into sensu:master Jan 10, 2018
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.

4 participants