<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/openstack/ironic-python-agent.git/ironic_python_agent/shell, branch stable/ocata</title>
<subtitle>opendev.org: openstack/ironic-python-agent.git
</subtitle>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/'/>
<entry>
<title>Fix for fatal error when GPT was used and only MBR was cleaned</title>
<updated>2018-05-19T18:14:41+00:00</updated>
<author>
<name>Olivier Bourdon</name>
<email>obourdon@opennext.io</email>
</author>
<published>2018-05-17T10:50:41+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=25cfa3a80a0a6204cc0c98996926cae972a10057'/>
<id>25cfa3a80a0a6204cc0c98996926cae972a10057</id>
<content type='text'>
You can generate this error if after having provisioned a node
using GPT partitioning, you clean its MBR using say
dd if=/dev/zero bs=1024 count=1 of=/dev/sda
and then cleanup all Ironic/Bifrost informations to get it
reprovisioned.
In this case sgdisk -Z returns an error and last_error field
in Ironic contains:
Error writing image to device: Writing image to device
        /dev/sda failed with exit code 2
Caution: invalid main GPT header, but valid backup;
        regenerating main header\nfrom backup!\n
        \nInvalid partition data!\

Change-Id: Ib617737fff5e40cb376edda0232e0726d9c71231
(cherry picked from commit 5fcf3f066ccdbcc081ae35ab25f1c18737b94778)
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
You can generate this error if after having provisioned a node
using GPT partitioning, you clean its MBR using say
dd if=/dev/zero bs=1024 count=1 of=/dev/sda
and then cleanup all Ironic/Bifrost informations to get it
reprovisioned.
In this case sgdisk -Z returns an error and last_error field
in Ironic contains:
Error writing image to device: Writing image to device
        /dev/sda failed with exit code 2
Caution: invalid main GPT header, but valid backup;
        regenerating main header\nfrom backup!\n
        \nInvalid partition data!\

Change-Id: Ib617737fff5e40cb376edda0232e0726d9c71231
(cherry picked from commit 5fcf3f066ccdbcc081ae35ab25f1c18737b94778)
</pre>
</div>
</content>
</entry>
<entry>
<title>Limit memory usage when running qemu-img convert</title>
<updated>2017-01-19T18:01:30+00:00</updated>
<author>
<name>Lucas Alvares Gomes</name>
<email>lucasagomes@gmail.com</email>
</author>
<published>2017-01-19T17:16:38+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=449cda4fed51250992446c83a4b50bc45fd10c6d'/>
<id>449cda4fed51250992446c83a4b50bc45fd10c6d</id>
<content type='text'>
This patch is limiting the memory address space to 1 GiB when invoking
qemu-img convert. The qemu-img command can be exploited so we better
play on the safe side (See: bug #1449062).

Change-Id: I2088f74b32d1392310c569cfaa7fd368cb737421
Closes-Bug: #1657808
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch is limiting the memory address space to 1 GiB when invoking
qemu-img convert. The qemu-img command can be exploited so we better
play on the safe side (See: bug #1449062).

Change-Id: I2088f74b32d1392310c569cfaa7fd368cb737421
Closes-Bug: #1657808
</pre>
</div>
</content>
</entry>
<entry>
<title>Moving Reboot bashscript to python</title>
<updated>2016-12-15T00:05:03+00:00</updated>
<author>
<name>Bharath kumar</name>
<email>shettybharath4@gmail.com</email>
</author>
<published>2016-09-19T12:41:04+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=9948349b10b6bb694f92813243598a7a7c84d3f8'/>
<id>9948349b10b6bb694f92813243598a7a7c84d3f8</id>
<content type='text'>
Currently a reboot bash script file is used to call reboot and
poweroff operation. Deleting this file and moving the code to
python file using utils.execute()

Partial-Bug: #1557542

Change-Id: Iad9cd9d15417e9a954d108d2759e6303452fca27
Author: Bharath kumar &lt;shettybharath4@gmail.com&gt;
Co-Authored-By: Annie Lezil &lt;annie.lezil@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Currently a reboot bash script file is used to call reboot and
poweroff operation. Deleting this file and moving the code to
python file using utils.execute()

Partial-Bug: #1557542

Change-Id: Iad9cd9d15417e9a954d108d2759e6303452fca27
Author: Bharath kumar &lt;shettybharath4@gmail.com&gt;
Co-Authored-By: Annie Lezil &lt;annie.lezil@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>Add bash scripts style checking for IPA</title>
<updated>2016-11-09T22:14:40+00:00</updated>
<author>
<name>OpenStack Proposal Bot</name>
<email>openstack-infra@lists.openstack.org</email>
</author>
<published>2016-11-04T14:08:31+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=c40575e4e046904d5dbd28826bf5308915093f15'/>
<id>c40575e4e046904d5dbd28826bf5308915093f15</id>
<content type='text'>
Updated file tox.ini allows to check code style in all ".sh" files
included in Ironic Python Agent. Checking can be invoked by calling either
"tox" or "tox -e pep8".

Change-Id: Ie76605737c7db10a064f2aebfda388372a4c0591
Closes-Bug: #1625215
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Updated file tox.ini allows to check code style in all ".sh" files
included in Ironic Python Agent. Checking can be invoked by calling either
"tox" or "tox -e pep8".

Change-Id: Ie76605737c7db10a064f2aebfda388372a4c0591
Closes-Bug: #1625215
</pre>
</div>
</content>
</entry>
<entry>
<title>Use ironic-lib to create configdrive</title>
<updated>2016-10-21T03:39:06+00:00</updated>
<author>
<name>Shivanand Tendulker</name>
<email>stendulker@gmail.com</email>
</author>
<published>2016-03-23T14:26:32+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=3665306dfbf9afadf06fc669b75a74e1c08b6b01'/>
<id>3665306dfbf9afadf06fc669b75a74e1c08b6b01</id>
<content type='text'>
Shell script to create config drive being replaced with python
code in ironic-lib.

Closes-Bug: #1493328

Change-Id: I31108f1173db3fb585386b2949ec880a95305fb6
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Shell script to create config drive being replaced with python
code in ironic-lib.

Closes-Bug: #1493328

Change-Id: I31108f1173db3fb585386b2949ec880a95305fb6
</pre>
</div>
</content>
</entry>
<entry>
<title>Fix config drive writing script</title>
<updated>2016-10-17T21:28:04+00:00</updated>
<author>
<name>Vladyslav Drok</name>
<email>vdrok@mirantis.com</email>
</author>
<published>2016-10-13T12:28:04+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=7bda3408f5bef4349c4e3537f3c8c0826a5dead8'/>
<id>7bda3408f5bef4349c4e3537f3c8c0826a5dead8</id>
<content type='text'>
When the deployment happens on a GPT disk with config drive, writing
it to disk fails. Three reasons for that:

* parted should be used instead of partprobe to determine the type of
  the disk;
* gdisk -l sorts the partitions by their number, sort them by start
  sector instead;
* after sgdisk completion, the configdrive device is not immediately
  visible in the /dev folder, udevadm settle needed here too.

Closes-Bug: #1633063
Change-Id: Ifed89e343f9db4cf303baf7f8823342f6041f202
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When the deployment happens on a GPT disk with config drive, writing
it to disk fails. Three reasons for that:

* parted should be used instead of partprobe to determine the type of
  the disk;
* gdisk -l sorts the partitions by their number, sort them by start
  sector instead;
* after sgdisk completion, the configdrive device is not immediately
  visible in the /dev folder, udevadm settle needed here too.

Closes-Bug: #1633063
Change-Id: Ifed89e343f9db4cf303baf7f8823342f6041f202
</pre>
</div>
</content>
</entry>
<entry>
<title>Prevent failure on 'copy_configdrive_to_disk.sh'</title>
<updated>2016-05-17T21:09:03+00:00</updated>
<author>
<name>Ricardo Araújo Santos</name>
<email>ricardo@lsd.ufcg.edu.br</email>
</author>
<published>2016-05-16T19:10:08+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=a90ee859a2c641c9461e918213c9816f53358548'/>
<id>a90ee859a2c641c9461e918213c9816f53358548</id>
<content type='text'>
The script splits 'sfdisk' output wrongly in blank spaces leading to
failures when partition id is greater than 10, e.g.:
"/dev/sda10: start= ...".

This fix splits the output on ":" and trims trailing space when
partition id is less than 10, e.g.: "/dev/sda1 : start= ...".

Change-Id: I23f4b747fc0a86713cb912afc5d193398e6a597b
Closes-Bug: 1581699
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The script splits 'sfdisk' output wrongly in blank spaces leading to
failures when partition id is greater than 10, e.g.:
"/dev/sda10: start= ...".

This fix splits the output on ":" and trims trailing space when
partition id is less than 10, e.g.: "/dev/sda1 : start= ...".

Change-Id: I23f4b747fc0a86713cb912afc5d193398e6a597b
Closes-Bug: 1581699
</pre>
</div>
</content>
</entry>
<entry>
<title>Replace SYSRQ commands</title>
<updated>2016-03-21T12:57:50+00:00</updated>
<author>
<name>Lucas Alvares Gomes</name>
<email>lucasagomes@gmail.com</email>
</author>
<published>2016-03-15T11:15:59+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=cbd90c1c14e083c5cc563cfd94f55266d28f77be'/>
<id>cbd90c1c14e083c5cc563cfd94f55266d28f77be</id>
<content type='text'>
This patch is modifying the shutdown.sh script to not use SYSRQ when
performing power actions. SYSRQ has a similar effect to a hardware reset
button/power switch, which can be problematic after copying an image to
the disk (data loss) therefore, this patch is changing the script to use
soft power action via the poweroff and reboot commands.

SYSRQ was required before because the ironic-python-agent service was
running inside a docker container, but since we've moved it to run into
a chroot (when built with coreos) or on the ramdisk's OS (when built
with DIB) the SYSRQ actions are not needed anymore.

In many parts of Ironic we were considering the standby.{power_off,
reboot} methods of IPA to be soft actions, which they are not at
present. This patch is also fixing that assumption.

Change-Id: Ibe3a3f61a4a0e98b2c2501ad95154839933b4d7a
Closes-Bug: #1557450
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch is modifying the shutdown.sh script to not use SYSRQ when
performing power actions. SYSRQ has a similar effect to a hardware reset
button/power switch, which can be problematic after copying an image to
the disk (data loss) therefore, this patch is changing the script to use
soft power action via the poweroff and reboot commands.

SYSRQ was required before because the ironic-python-agent service was
running inside a docker container, but since we've moved it to run into
a chroot (when built with coreos) or on the ramdisk's OS (when built
with DIB) the SYSRQ actions are not needed anymore.

In many parts of Ironic we were considering the standby.{power_off,
reboot} methods of IPA to be soft actions, which they are not at
present. This patch is also fixing that assumption.

Change-Id: Ibe3a3f61a4a0e98b2c2501ad95154839933b4d7a
Closes-Bug: #1557450
</pre>
</div>
</content>
</entry>
<entry>
<title>Clear GPT and MBR data structures on disk before imaging</title>
<updated>2016-02-09T17:13:07+00:00</updated>
<author>
<name>Shivanand Tendulker</name>
<email>stendulker@gmail.com</email>
</author>
<published>2016-02-09T16:35:36+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=8f5ed3e2f16cd401b41b2c27617251e8d9894cf2'/>
<id>8f5ed3e2f16cd401b41b2c27617251e8d9894cf2</id>
<content type='text'>
This patch clears all GPT and MBR data structures on the disk.
GPT maintains its data structures at start and at end of the
disk.

Change-Id: I8d0b367755b568cb44f02f62b873659b4bea0f14
Closes-Bug: #1543339
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This patch clears all GPT and MBR data structures on the disk.
GPT maintains its data structures at start and at end of the
disk.

Change-Id: I8d0b367755b568cb44f02f62b873659b4bea0f14
Closes-Bug: #1543339
</pre>
</div>
</content>
</entry>
<entry>
<title>Add tinyipa to IPA imagebuild directory</title>
<updated>2016-01-29T12:07:19+00:00</updated>
<author>
<name>Sam Betts</name>
<email>sam@code-smash.net</email>
</author>
<published>2015-10-13T15:29:40+00:00</published>
<link rel='alternate' type='text/html' href='http://91.123.203.49/cgit/delta/openstack/ironic-python-agent.git/commit/?id=632c7e6d31f00409a0e9268125496471c8ed3aa3'/>
<id>632c7e6d31f00409a0e9268125496471c8ed3aa3</id>
<content type='text'>
TinyCoreLinux is a small 10mb base linux distribution which can easily
be built into a ramdisk for PXE and ISO booting. This patch adds scripts
and other required files to build a TinyCore based ramdisk which runs
the Ironic Python Agent.

Change-Id: I79fbec75de19b3d6feb19f87a3a0f662eae42bf3
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
TinyCoreLinux is a small 10mb base linux distribution which can easily
be built into a ramdisk for PXE and ISO booting. This patch adds scripts
and other required files to build a TinyCore based ramdisk which runs
the Ironic Python Agent.

Change-Id: I79fbec75de19b3d6feb19f87a3a0f662eae42bf3
</pre>
</div>
</content>
</entry>
</feed>
