/layer-cwr

Adding build badge #71

Merged
merged 5 commits into from Feb 14, 2017

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@ktsakalozos @johnsca @kwmonroe
@ktsakalozos
Member

ktsakalozos commented Feb 13, 2017

No description provided.

ktsakalozos and others added some commits Feb 13, 2017

@ktsakalozos @johnsca
Adding build badge
50565aa
@johnsca
Make Jenkins job name match CWR
88fd558
@johnsca
Restart server after upgrade
4740b70
@johnsca
Use dynamic badge template 
The badge will now show the results for each individual controller,
rather than the total build pass / fail.

The URL of the badge was simplified as well.

All checks have failed

2 errored checks
continuous-integration/travis-ci/pr — The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push — The Travis CI build could not complete due to an error
Details
809f59e
@johnsca
Owner

johnsca commented Feb 14, 2017

Updated to use a dynamic badge template. Now, instead of getting a single pass / fail result, we will get info about each controller:

badge

One side-effect of this is that the Jenkins job names were changed so that they match up with the CWR name, both reported and on disk. I also removed the REST API prefix from the badge URL because it's more friendly and it's not truly an API endpoint.
@ktsakalozos
Member

ktsakalozos commented Feb 14, 2017

I did not have much luck with testing this PR.

Traceback (most recent call last):
  File "/var/lib/juju/agents/unit-cwr5-0/charm/scripts/bundlebuilder.py", line 13, in <module>
    from actions.cwrhelpers import get_fname
ImportError: No module named 'actions'

And then unit tests fail as well.
@johnsca
Fix import error and tests

All checks have failed

2 errored checks
continuous-integration/travis-ci/pr — The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push — The Travis CI build could not complete due to an error
Details
a4473d3
@kwmonroe
Member

kwmonroe commented Feb 14, 2017

Looks GREAT to me!

@kwmonroe kwmonroe merged commit c4c3506 into master Feb 14, 2017
0 of 2 checks passed

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/travis-ci/push The Travis CI build could not complete due to an error
Details

@kwmonroe kwmonroe deleted the feature/build-badge branch Feb 14, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment