Merge ~redriver/cloud-init:fail_to_install_on_frbsd into cloud-init:master

Proposed by Hongjiang Zhang
Status: Merged
Merge reported by: Scott Moser
Merged at revision: 3ac3e560626348e3efd88b30341e3e433b961a9e
Proposed branch: ~redriver/cloud-init:fail_to_install_on_frbsd
Merge into: cloud-init:master
Diff against target: 35 lines (+2/-4)
2 files modified
cloudinit/util.py (+1/-1)
tools/build-on-freebsd (+1/-3)
Reviewer Review Type Date Requested Status
Server Team CI bot continuous-integration Approve
cloud-init Commiters Pending
Review via email: mp+325718@code.launchpad.net

Commit message

tools/build-on-freebsd was blocked

fail to run tools/build-on-freebsd on FreeBSD platform.

LP: #1698077

Description of the change

tools/build-on-freebsd was blocked

fail to run tools/build-on-freebsd on FreeBSD platform.

LP: #1698077

To post a comment you must log in.
Revision history for this message
Server Team CI bot (server-team-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Scott Moser (smoser) wrote :

I'll look at this later, but i'm not sure we can just
- 'platform': platform.platform(),
+ 'platform': platform.uname()[0],

without breaking other things. I'll try this out today though and will find a freebsd to test on.

Revision history for this message
Scott Moser (smoser) :
Revision history for this message
Scott Moser (smoser) wrote :
Revision history for this message
Scott Moser (smoser) wrote :

I'm marking this merged as the intent of it is merged.
the merge proposal above is in, and I think you should be good for building on freebsd now.

Revision history for this message
Hongjiang Zhang (redriver) wrote :

Yes. The build is successful on FreeBSD. Thanks

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/cloudinit/util.py b/cloudinit/util.py
2index 415ca37..2c8f04f 100644
3--- a/cloudinit/util.py
4+++ b/cloudinit/util.py
5@@ -597,7 +597,7 @@ def get_cfg_option_int(yobj, key, default=0):
6
7 def system_info():
8 info = {
9- 'platform': platform.platform(),
10+ 'platform': platform.uname()[0],
11 'release': platform.release(),
12 'python': platform.python_version(),
13 'uname': platform.uname(),
14diff --git a/tools/build-on-freebsd b/tools/build-on-freebsd
15index ccc10b4..f16d62e 100755
16--- a/tools/build-on-freebsd
17+++ b/tools/build-on-freebsd
18@@ -17,6 +17,7 @@ pkgs="
19 py27-jsonpatch
20 py27-jsonpointer
21 py27-oauth
22+ py27-oauthlib
23 py27-prettytable
24 py27-requests
25 py27-serial
26@@ -35,9 +36,6 @@ touch $depschecked
27 python setup.py build
28 python setup.py install -O1 --skip-build --prefix /usr/local/ --init-system sysvinit_freebsd
29
30-# Install the correct config file:
31-cp config/cloud.cfg-freebsd /etc/cloud/cloud.cfg
32-
33 # Enable cloud-init in /etc/rc.conf:
34 sed -i.bak -e "/cloudinit_enable=.*/d" /etc/rc.conf
35 echo 'cloudinit_enable="YES"' >> /etc/rc.conf

Subscribers

People subscribed via source and target branches