Merge ~mthaddon/charm-k8s-discourse/+git/charm-k8s-discourse:charmcraft-yaml into charm-k8s-discourse:master

Proposed by Tom Haddon
Status: Merged
Merged at revision: c6cb23e62b03e1253f9fffcc3103d551c50e12d8
Proposed branch: ~mthaddon/charm-k8s-discourse/+git/charm-k8s-discourse:charmcraft-yaml
Merge into: charm-k8s-discourse:master
Diff against target: 61 lines (+10/-9)
3 files modified
Makefile (+1/-6)
charmcraft.yaml (+8/-0)
tox.ini (+1/-3)
Reviewer Review Type Date Requested Status
🤖 prod-jenkaas-is (community) continuous-integration Needs Fixing
Canonical IS Reviewers Pending
Discourse Charm Maintainers Pending
Review via email: mp+411514@code.launchpad.net

Commit message

Add a charmcraft.yaml and update path for tox

To post a comment you must log in.
Revision history for this message
🤖 prod-jenkaas-is (prod-jenkaas-is) wrote :

A CI job is currently in progress. A follow up comment will be added when it completes.

Revision history for this message
🤖 prod-jenkaas-is (prod-jenkaas-is) wrote :

FAILED: Continuous integration, rev:bb013da16c36c83dfa103d472fe21eac40d84a94

No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want jenkins to rebuild you need to trigger it yourself):
https://code.launchpad.net/~mthaddon/charm-k8s-discourse/+git/charm-k8s-discourse/+merge/411514/+edit-commit-message

https://jenkins.canonical.com/is/job/lp-charm-k8s-discourse-ci/34/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/is/job/lp-charm-test/149/
    None: https://jenkins.canonical.com/is/job/lp-update-mp/538387/

Click here to trigger a rebuild:
https://jenkins.canonical.com/is/job/lp-charm-k8s-discourse-ci/34//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Unable to determine commit message from repository - please click "Set commit message" and enter the commit message manually.

Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Failed to merge change (unable to merge source repository due to conflicts), setting status to needs review.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/Makefile b/Makefile
index d754dab..f4d845e 100644
--- a/Makefile
+++ b/Makefile
@@ -10,9 +10,7 @@ lint: blacken
10 @echo "Running flake8"10 @echo "Running flake8"
11 @tox -e lint11 @tox -e lint
1212
13# We actually use the build directory created by charmcraft,13unittest:
14# but the .charm file makes a much more convenient sentinel.
15unittest: discourse-k8s.charm
16 @tox -e unit14 @tox -e unit
1715
18test: lint unittest16test: lint unittest
@@ -21,9 +19,6 @@ clean:
21 @echo "Cleaning files"19 @echo "Cleaning files"
22 @git clean -fXd20 @git clean -fXd
2321
24discourse-k8s.charm: src/*.py requirements.txt
25 charmcraft build
26
27build-image:22build-image:
28 @echo "Building the image."23 @echo "Building the image."
29 @docker build \24 @docker build \
diff --git a/charmcraft.yaml b/charmcraft.yaml
30new file mode 10064425new file mode 100644
index 0000000..bf04cb1
--- /dev/null
+++ b/charmcraft.yaml
@@ -0,0 +1,8 @@
1type: charm
2bases:
3 - build-on:
4 - name: "ubuntu"
5 channel: "20.04"
6 run-on:
7 - name: "ubuntu"
8 channel: "20.04"
diff --git a/tox.ini b/tox.ini
index d48b030..3d6bf55 100644
--- a/tox.ini
+++ b/tox.ini
@@ -5,8 +5,6 @@ skip_missing_interpreters = True
55
6[testenv]6[testenv]
7basepython = python37basepython = python3
8setenv =
9 PYTHONPATH = {toxinidir}/build/lib:{toxinidir}/build/venv
108
11[testenv:unit]9[testenv:unit]
12commands =10commands =
@@ -15,7 +13,7 @@ commands =
15deps = -r{toxinidir}/tests/unit/requirements.txt13deps = -r{toxinidir}/tests/unit/requirements.txt
16 -r{toxinidir}/requirements.txt14 -r{toxinidir}/requirements.txt
17setenv =15setenv =
18 PYTHONPATH={toxinidir}/src:{toxinidir}/build/lib:{toxinidir}/build/venv16 PYTHONPATH = {toxinidir}/lib:{toxinidir}/src
19 TZ=UTC17 TZ=UTC
2018
21[testenv:black]19[testenv:black]

Subscribers

People subscribed via source and target branches