unconfigre_tenant_networks() removes vif_port_id from ironic port,
as result during deploy we don't know for which port to update DHCP.
It was reworked in Ocata, but shouldn't be present in Newton.
Related-Bug: #1675673
Change-Id: I3a2da0ab20ed32273c8e426c0d13b0fd02583a54
The contents of /etc/nailgun_uid can be changed by nailgun agent. We need to
synchronize the change to mcollective config.
Closes-Bug: #1675668
Change-Id: Idabc7be6181bc89c09e4d0caa4a7cf0c22767bfe
(cherry picked from commit ea8fc2bdfe)
All packages required for fuel-nailgun added to the bootstrap image.
Dask worker startup added.
Change-Id: I7155601a0f95c891a40f3c9d02d1758936ffbb45
Implements: blueprint distributed-serialization
This patch adds logic to work with network providers to fuel_* ironic drivers.
Depends-On: I742a52c991058d1f8bd9170f9b583c28dff22839
Change-Id: Ic3f831a9a9e5ec9031bf1a588df62c9338c47d20
Related-Bug: #1588380
Network checker should be able to run on bootstrap nodes
without the access to repositories.
Closes-bug: #1653294
Change-Id: I4fae538968ab4a75269858a73e5065818a9f4f04
(cherry picked from commit 9488b8a324)
In our usecases the separate partition is not needed. It is enough just
to put cloudinit configuration into the root filesystem.
This also allows to avoid a race condition which sometimes happens: some
process deletes the folder in tmp where the configuration partition is
mounted resulting in cloudinit failure to read its configuration.
Change-Id: Ib3efb4f517a5cf86dbf91ee53ac00108d4624dcd
Closes-Bug: #1652002
(cherry picked from commit b9842ce714)
Add a special rule for fcoe devices to reuse ID and net_name
to avoid renaming issues in bootstrap.
Change-Id: I40c699b37199f93bfdf0157d0eb795b735254990
Closes-bug: #1656892
(cherry picked from commit 9ea6419878)
As we known, Exceptions are raised by the sys.exit() function.
When they are not handled, no stack traceback is printed in the Python interpreter.
In this patch we have known main() got return values(eg:0, 1), but it can't
specifies exit status when terminate the main thread by unusing sys.exit().
So when using sys.exit(main()) instead of main() may be more readable and
reasonable.
Change-Id: I6b472e0eb2fceb49bb506785f5188e023e1d3968
(cherry picked from commit 675eb6abaf)
As a part of handing over control over mcollective from nailgun agent to
startup scripts lets get rid of of nodiscover file flag and perform
mcollective configuration and start
Change-Id: Ia2f984570b38642b1090f6483ed3fa78958550c5
Partial-Bug: #1585671
(cherry picked from commit 5877315c7f)
Nobody promised PATH to be set during rc.local execution by systemd. And
it happens that this is the case on some systems, i.e. the PATH is not
set by the time of rc.local execution.
Change-Id: I4806df3217dd09430ad45b13cda49fab0925cb88
Partial-Bug: #1656150
(cherry picked from commit 5c612875a6)
Refer to:http://docs.openstack.org/developer/hacking/#unit-tests-and-assertraises
[H203] Use assertIs(Not)None to check for None (off by default) Unit
test assertions tend to give better messages for more specific assertions.
As a result, assertIsNone(...) is preferred over assertEqual(None, ...)
and assertIs(None, ...)
Change-Id: I9246fac952c59d3ddc1458c16c53fc988ac095d3
(cherry picked from commit 9e1e030c98)
Those deleted kernel parameters are extremely heavy debug.
The boot process is rather slow on machines which has a
complicate motherboard and lots of pci devices(Lenovo RD640).
The debug process costs so much resouce, evetually it will impact
normal functions, such as cpus are so busy dealing with debug
info in kernelspace, they do not have time to reponse udev
event in userspace and failed to bring up pci devices.
Change-Id: I964c5fbc7f5b7d0ba6d1dd0166db3e4234179afa
Closes-bug: #1632592
(cherry picked from commit babe668c21)
It seems that there are at least two level of ruby-stomp brokenness and
the fact that the mutex in original commit is locked actually means
there is no heartbeat received in time and we need to do something about
this.
This reverts commit b50241a7b2.
Closes-Bug: #1644618
Change-Id: I8351abaf0078b094bff2aa20994575c15aec213b
(cherry picked from commit f15987c3f2)
Such services as Swift-storage require partition with file
system without mountpoints, currently fuel-agent prevents
to do that.
Change-Id: Id9e90f81098de3736e0d9b1ca82e434122efd4e8
Closes-bug: #1650622
(cherry picked from commit d9f9f377e4)
The issue is that in python-fuelclient API has been
changed and fuelclient.client.APIClient is not available
anymore. It used to be a singleton of fuel client but
now it is deprecated. So, let's use APIClient.default_client
classmethod to instantiate APIClient.
https://github.com/openstack/python-fuelclient/blob/stable/newton/fuelclient/client.py#L51
Change-Id: I5287184c6f1c6abf8ac61894db0bd51fbf730401
Closes-Bug: #1620584
The previos implementation doesn't fix the problem. Cloud-init
creates /etc/network/interfaces.d/50-cloud-init.cfg which prevents
to set static IP.
Change-Id: I49d09dd37403a843adfe34be156e5e265b0f3e08
Signed-off-by: Sergii Golovatiuk <sgolovatiuk@mirantis.com>
(cherry picked from commit fa49edcdae)
This fix doesn't fix the problem. I still see 50-cloud-init.cfg in /etc/network/interfaces.d/
This reverts commit faa9de1780.
Change-Id: I93a7fa29ae932b0020a4dad17e580808ee3fb117
(cherry picked from commit 51d3078157)
Stomp heartbeat handling is quite poorly designed. It happens in a
separate thread which sleeps, then tries to read a heartbeat if reading
mutex is acquired by message receiving thread it fails and increases
lock failure count. Upon reaching the limit (in our packets it is 2 by
default) it forcibly closes the connetion causing reconnect. Setting the
value to 0 turns the feature off.
Change-Id: I2187ce69508c530073582c542c963014acc5123a
Closes-Bug: #1613246
Closes-Bug: #1298262
Config fuel_bootstrap_cli.yaml should be updated along with
fuel-bootstrap-cli package update. This config will be modified
later when master node will be configured: http://goo.gl/W36lG4
Change-Id: I4861e429a121b45aa864ec63fe164a03cf2ed74b
Closes-bug: #1620301
This patch set modifies lines which are importing objects
instead of modules. As per openstack import guide lines, user should
import modules in a file not objects.
http://docs.openstack.org/developer/hacking/#imports
Change-Id: Icfd86bfff12889e76316da6788db4f7f0c352275
by default, Ironic's API routing now exposes only 'lookup' and 'heartbeat'
vendor passthru methods of drivers as public routes [0].
Rename the vendor passtru method 'pass_deploy_info' of fuel-agent driver
to 'heartbeat' to make it accessable w/o authentication by callback
script.
In the Ocata cycle, this also will be removed and fuel-agent driver
must switch to the root 'heartbeat' API (/v1/heartbeat/node-uuid>).
[0]
https://github.com/openstack/ironic/blob/master/ironic/api/config.py#L37
Change-Id: Ie1dbd8a8aac62034beba88ecd206a8e7712514e2
The origin message format is simply a coding mistake. Users will get
useless error message since we didn't really passing correct `url` in
error message. This patch fix HttpUrlConnectionError error message
format.
Closes-Bug: #1614048
Change-Id: I5b9b5dcfca053097622ea6b701aa758ee1a5aa8e
Provision process moving to fuel-library tasks from nailgun and astute
and as all tasks we have are wrapped to puppet manifests, we need
puppet to have an ability running provision tasks in bootstrapped node.
Implements: blueprint graph-concept-extension
Change-Id: I0f0078071f8e32bace345d636c602e68c91aa55b
Having admin NIC confugured in the middle of system booting
is very fragile and error-prone approach.
It's better to configure it ahead of first booting time.
Also, there's no need for freaky networking restarting inside of
cloud-init boothooks anymore.
Change-Id: I4c278341e8b40eb8d9b100bed1d9a650f27b5c5b
Related-Bug: #1583815