Commit Graph

551 Commits

Author SHA1 Message Date
OpenDev Sysadmins a6d20b1408 OpenDev Migration Patch
This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:

http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html
http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html

Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.
2019-04-19 19:34:29 +00:00
Jenkins e2c0e2744b Merge "Using sys.exit(main()) instead of main()" into stable/newton 2017-04-25 11:59:56 +00:00
Vladimir Kozhukalov 9b15d47ad5 Add udevadm settle after setting flags
This is not going to make things worse.

Change-Id: I9fb6cf5e6b6f64da8f1aaf8611840b6a108acb49
Closes-Bug: #1675734
2017-03-29 20:17:54 +03:00
Vasyl Saienko 75e69a0f99 Drop unconfigure_tenant_networks in prepare
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
2017-03-29 10:10:18 +00:00
Alexander Kislitsky 1aa048ec6a Revert "Packages for distributed serialization added to bootstrap"
This reverts commit d339d77439.

Change-Id: I9e734f0fd9b847630923acfd6e57635abd4e366f
2017-03-27 19:39:34 +03:00
Georgy Kibardin 561e1f5ba2 Sync mcollective config with /etc/nailgun_uid
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)
2017-03-25 11:19:08 +03:00
Alexander Kislitsky d339d77439 Packages for distributed serialization added to bootstrap
All packages required for fuel-nailgun added to the bootstrap image.
Dask worker startup added.

Change-Id: I7155601a0f95c891a40f3c9d02d1758936ffbb45
Implements: blueprint distributed-serialization
2017-03-24 14:08:33 +03:00
Jenkins 8a17d5d303 Merge "[Multitenancy] Add logic to work with network providers" into stable/newton 2017-03-23 15:55:58 +00:00
Vasyl Saienko 0ecbf5843a [Multitenancy] Add logic to work with network providers
This patch adds logic to work with network providers to fuel_* ironic drivers.

Depends-On: I742a52c991058d1f8bd9170f9b583c28dff22839

Change-Id: Ic3f831a9a9e5ec9031bf1a588df62c9338c47d20
Related-Bug: #1588380
2017-03-23 13:27:52 +00:00
Georgy Kibardin be14663f87 Fix requests dependency
Change-Id: I742a52c991058d1f8bd9170f9b583c28dff22839
(cherry picked from commit 6288a593e2)
2017-03-23 11:52:07 +00:00
Jenkins bb9ece4266 Merge "Fix LOG.warn to LOG.warning" into stable/newton 2017-03-09 11:26:18 +00:00
Evgeny L 22f0e4cbf6 Add to bootstrap image dependencies for network checker
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)
2017-03-02 11:11:06 +00:00
Jenkins 2dc6de21ea Merge "Add FCoE rules to fix env renaming" into stable/newton 2017-02-21 14:19:49 +00:00
Georgy Kibardin 739326df02 Do not use separate partition for cloudinit configuration
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)
2017-02-21 11:24:53 +00:00
azvyagintsev a3efbff41f Disable Recommends and Suggests packages
Disable Recommends and Suggests package installs,
  inside Ubuntu target-image

Change-Id: Ie8a450a960b9b00cd4d2c0545a19678baed147d6
Co-Authored-By: Ivan Suzdal <isuzdal@mirantis.com>
Closes-Bug: 1664152
(cherry picked from commit 6f6fd2fa42)
2017-02-17 13:07:51 +00:00
azvyagintsev 99b0cd6b3e Add rsync dependency for fuel-agent
Change-Id: I05e0b33420058772216220bb9fcd5f61c40f3e27
Co-Authored-By: Ivan Suzdal <isuzdal@mirantis.com>
Partial-Bug: 1664152
2017-02-16 10:37:07 +00:00
Jenkins 336d5ccadd Merge "Use assertIsNone(...) instead of assertEqual(None, ...)" into stable/newton 2017-02-15 13:38:06 +00:00
Vladimir Kuklin c82fbe07b0 Add FCoE rules to fix env renaming
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)
2017-02-13 16:17:12 +00:00
gecong1973 5e78a69966 Using sys.exit(main()) instead of main()
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)
2017-02-13 16:14:20 +00:00
Georgy Kibardin 16318a98eb Configure mcollective identity in startup scripts
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)
2017-02-13 10:59:46 +03:00
Georgy Kibardin 9526f477d8 Use explicit path in Xenial
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)
2017-02-09 09:51:36 +00:00
Cady_Chen 6c9f614fc0 Use assertIsNone(...) instead of assertEqual(None, ...)
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)
2017-02-08 12:54:49 +00:00
Cao Xuan Hoang d010708d92 Fix LOG.warn to LOG.warning
logging.warn is deprecated in Python 3.
https://docs.python.org/3/library/logging.html#logging.warning

Change-Id: I5cef5a11e4802d9244295e2c871a848d14ab7454
(cherry picked from commit 075fc9e257)
2017-01-27 15:54:28 +00:00
Wei Hui e950bee9ed Delete kernel debug related parameter for performance
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)
2017-01-09 01:29:43 +00:00
Georgy Kibardin c02f49f40d Revert "Ignore heartbeats lock fails"
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)
2016-12-28 11:52:58 +00:00
Evgeny L 08d578a81e Allow fs creation if mountpoint is not specified
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)
2016-12-20 16:45:10 +00:00
Vladimir Kozhukalov 02b7979845 Change the way how we instantiate fuel client
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
2016-12-14 16:25:47 +03:00
Jenkins 7d35078a9d Merge "Revert "Disable cloud-init network auto configuration"" into stable/newton 2016-11-23 10:25:15 +00:00
Sergii Golovatiuk 1d1d27c30f Disable cloud-init network auto configuration
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)
2016-11-19 19:03:29 +00:00
Sergii Golovatiuk 2e3b0ad4b6 Revert "Disable cloud-init network auto configuration"
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)
2016-11-19 13:07:52 +01:00
Vladimir Kozhukalov d70c55831a Add pbr version hook to fuel_bootstrap
Change-Id: I5f3778c86314c59d90a8f1b74a4aa1bc8e90241e
Closes-Bug: #1642549
(cherry picked from commit 0fd70bb712)
2016-11-18 17:58:28 +00:00
Davanum Srinivas 4d5f9e9467 Update .gitreview for stable/newton
Change-Id: Ib03801dedf22673ff527e581eb8161ffc0c5f51c
2016-10-14 09:14:55 -04:00
Vasyl Saienko 8125d9ecd5 Add ironic-callback systemd script.
This patch replaced ironic-callback upstart script by systemd.

Change-Id: Idfbe2de940f0e99e8d0fcf952a9153631b6789b1
Closes-Bug: #1624011
2016-09-16 15:52:36 +03:00
Jenkins 850606843a Merge "Change an order of executing cloud-init modules" 2016-09-15 12:21:06 +00:00
Jenkins 907b7962f2 Merge "Ignore heartbeats lock fails" 2016-09-14 12:58:01 +00:00
Jenkins 62a7800e26 Merge "Configure admin NIC prior the first system boot" 2016-09-14 12:52:52 +00:00
Georgy Kibardin b50241a7b2 Ignore heartbeats lock fails
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
2016-09-13 18:46:21 +03:00
Mikhail 52c4cfd937 Change an order of executing cloud-init modules
Module 'write-files' should be executed after module 'runcmd'

Change-Id: I93360afb05ab739fe3f44b187db41054673caa0e
Closes-Bug: #1543063
2016-09-13 17:34:27 +03:00
iberezovskiy 8234a29665 Replace fuel_bootstrap_cli.yaml on package installation/update
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
2016-09-07 21:30:38 +00:00
Jenkins bcbb1ffc1d Merge "TrivialFix: Remove logging import unused" 2016-09-06 18:22:26 +00:00
Jenkins 5acbe3cf7d Merge "Clean imports in code" 2016-09-06 18:17:57 +00:00
Albert b3b10de4c5 Enabling specify --config file from CLI when building fuel-bootstrap
Closes-Bug: #1608925

Change-Id: I5bac895daf85bdc52b1672f53fec1b707db24473
2016-09-06 11:33:49 +00:00
Cao Xuan Hoang 00533687d9 Clean imports in code
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
2016-09-06 13:26:32 +07:00
Cao Xuan Hoang 6c206f7838 TrivialFix: Remove logging import unused
This patch removes logging import unused in
fuel_agent/objects/partition/parted.py

Change-Id: Ia14a5a916ea37c626371467b654c44188e0149d0
2016-09-06 13:05:12 +07:00
Pavlo Shchelokovskyy cb77827303 Rename pass_deploy_info to heartbeat
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
2016-08-31 16:51:15 +03:00
Jenkins ecd7a67ee0 Merge "Add puppet and daemonize to default packages list" 2016-08-24 16:21:23 +00:00
ricolin d80f7a0ba8 Fix HttpUrlConnectionError error message format
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
2016-08-19 05:56:23 +00:00
Stanislaw Bogatkin f88768583c Add puppet and daemonize to default packages list
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
2016-08-17 14:27:13 +00:00
Alexander Gordeev da7a578473 Configure admin NIC prior the first system boot
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
2016-08-16 18:29:09 +03:00
Dmitry Teselkin 426d2a39f4 Fix bootstrap apt-get update issues
Sometimes 'apt-get update' fail with strange error:

  Get:22 http://mirror.fuel-infra.org/mos-repos/ubuntu/10.0 mos10.0-updates Release.gpg [473 B]
  Err:22 http://mirror.fuel-infra.org/mos-repos/ubuntu/10.0 mos10.0-updates Release.gpg
    Writing more data than expected (473 > 20) [IP: 208.78.244.194 80]

This is only reproduced during running bootstrap, and never reproduced manually.

APT::http::Pipeline-Depth=0 fixes the problem.

By default pipeline depth is 10 in apt 1.2.10 (Ubuntu Xenial) [1], [2].

[1] https://github.com/Debian/apt/blob/1.2.10/doc/apt.conf.5.xml#L402-L407
[2] https://github.com/Debian/apt/blob/1.2.10/methods/server.cc#L754-L756

Related-bug: #1612731
Related-bug: #1470468

Change-Id: Iac8c22d87f18718ae9a318dda56266f8ed57f866
2016-08-15 15:43:10 +00:00