Commit Graph

889 Commits

Author SHA1 Message Date
Alex Schultz 87abe05ba0 Retire instack-undercloud
instack-undercloud is no longer in use by the TripleO project. Removing
the code to avoid confusion. Stable branches will continue to be
maintained for their life however no new features should be added.

Change-Id: I63a813c7c1ffd30ca30017133d31a497b77a9a4d
Blueprint: remove-instack-undercloud
2018-10-30 12:16:18 +00:00
Juan Antonio Osorio Robles 3a67a6c8cb Remove certificate resubmit from haproxy script
This was needed because puppet-certmonger didn't have the means to
request the appropriate EKU values manually (it relied on the defaults
per CA). This is no longer the case, so we can remove it.

Depends-On: Ie54d10fd4da45a73529a05e29aaa8eb19f986df3
Change-Id: I0eb466d7311e012d10fe03651c39dfd9a77bc517
2017-09-26 15:06:22 +03:00
Juan Antonio Osorio Robles fe25c53fe9 Undercloud/Certmonger: Only attempt to reload haproxy is it's active
Previously, certmonger tried to reload haproxy every time after a
certificate is requested. This is useful for certificate resubmits or
renewals. However, it turned out problematic on installation, when
haproxy is not yet active, as it would try many times and end up having
a race-condition with puppet.

This checks if haproxy is active and only then will it attempt to reload
it.

Closes-Bug: #1712377
Change-Id: I4edd42b888a0bbbb8eb0e71f5c17750bac46c2ce
2017-09-01 14:18:45 +00:00
Ben Nemec bda75f1640 Add pre-upgrade entrypoint
This will allow tripleoclient to update instack-undercloud, call the
new pre_upgrade hook which will contain any new steps needed for
the coming release, and then call the regular upgrade script to
apply all the new configuration.

bp undercloud-upgrade

Change-Id: If7c2f5f5a904d570b2f2fbe4eea3f8fd7162afb9
2017-06-29 21:39:46 +00:00
James Slagle 13d18699aa Remove instack-virt-setup
We deprecated this script in ocata and can now remove it. The depends on
patch updates tripleo-docs to point to tripleo-quickstart directly for
creating virtual environments.

Change-Id: I920c73a1eb9b83644d7dd7dc4768ef1bd9949cdd
2017-04-04 11:56:55 +00:00
Mark Hamzy 6b5e2b738c Add a pointer to the tripleo-quickstart project as a replacement
Change-Id: Ic660aba852b45fa21f13ef8ea563b10f1ed3c486
2017-03-30 10:40:37 -05:00
Dan Trainor 48b293dde6 Add certificate EKUs to public endpoint cert
Add EKUS, or Extended Key Usage parameters, of id-kp-clientAuth and
id-kp-serverAuth to the certificate that certmonge generates, which is
used by haproxy to proxy public-facing hosts.  This is necessary due to
the criteria by which Firefox and related browsers validate which
required extensions are acceptable when interpreting a certificate.

Change-Id: Ideec7d23769e68ae1b738c0118ec061b195e3bd7
Closes-Bug: 1668775
2017-03-02 09:59:41 -05:00
James Slagle 84f58a17fe Deprecate instack-virt-setup
Let's go ahead and deprecate instack-virt-setup in Ocata. Then we can
see about removing it in a future release.

Change-Id: I22de582de69c15eb74d4de09c3a2d7299aee8f3f
2017-01-25 14:08:13 -05:00
Alex Schultz a661b6d5e1 Improve upgrade process to include upgrade flag
Previously, the upgrade process for the undercloud has been to just
rerun the install process. The problem is that this does not include
anwyay to indicate that it's already been run if we need to trigger some
different actions on subsequent runs.  As part of the Newton to Ocata
upgrade, we need to be able to run the cell v2 setup in a different
order than on a traditional install. We need a way for the puppet
scripts that ultimately get run to know if it's an upgrade or install
action. This change adds the creation of an undercloud_upgrade fact that
will be used when the upgrade process is run. Additionally a new script
called instack-upgrade-undercloud has been created to be used when
upgrade. The use of instack-install-undercloud will cause the install
flow to be execuated, while the instack-upgrade-undercloud will set the
fact to true to allow for the puppet scripts to use the upgrade flow.

Change-Id: Ie3cb21e30334fe8ffc0a9d6e707b42269b64c9ec
Related-Bug: #1649341
Related-Blueprint: undercloud-upgrade
2017-01-17 15:44:02 -07:00
Emilien Macchi 9031dcf620 Fix bashate errors and warnings (lint)
Fix 1 warning and 1 error in bash syntax:

1) E043: Arithmetic compound has inconsistent return semantics.
   Fix the bash syntax to be compliant.

2) ERROR: Split export and assignments in 2 files. Ignore the files.
   Fix the bash syntax to be compliant.
   Note: this test has been introducted by
   https://review.openstack.org/#/c/400542/

Change-Id: I3c1134336033a8b5f2d56cd87fa190d9a354ecc0
2016-12-17 09:49:32 -05:00
Ben Nemec 1b89f5bf7b Remove instack-create-overcloudrc
This is now handled by tripleoclient, and has generated a bug
report.  Let's get rid of it.

Change-Id: I3c51b5d920efe26fa47c3ae033f2b3199dbb2eeb
Closes-Bug: 1634052
2016-10-19 18:43:44 +00:00
James Slagle cea03d4a5f Update default VM memory
Update instack-virt-setup to match the known minimum requirements for VM
memory (matches what tripleo-ci currently uses).

Change-Id: I435405d787134b1f0ea3e6bfef4b66faa9fde551
2016-09-22 14:44:09 -04:00
Juan Antonio Osorio Robles faeed2494e Use certmonger for automatic cert generation
This will enable us to use a real CA to request the service certificates.

bp tls-via-certmonger

Depends-On: I32ded4e33abffd51f220fb8a7dc6263aace72acd
Change-Id: I5009273110154f0327ad542d75e83ff67bf72613
2016-08-03 13:44:21 +03:00
Giulio Fidente 622511dd4f Allow customization of undercloud disk size
This will permit for customization of the undercloud disk size
via UNDERCLOUD_NODE_DISK variable.

Change-Id: I11f6715992b1ba79f73fc9d8b8f401213db373c2
2016-07-19 12:43:43 +02:00
Carlos Camacho 2dec7d7521 Overcloud is not able to deploy with the default 4GB of RAM using instack-undercloud
When deploying the overcloud with the default value of 4GB of RAM the overcloud fails throwing "Cannot allocate memory" errors.
By increasing the default memory to 5GB the error is solved in instack-undercloud

Change-Id: I29036edeebefc1959643a04c5396e72863fdca5f
Closes-Bug: #1563750
2016-03-30 09:17:44 +00:00
James Slagle c2d5c0f862 Remove unused scripts
All these scripts are replaced by python-tripleoclient or manual docs
(in the tempest case). For better or worse, I don't think a deprecation
period is necessary here as I think we're way past that point with the
reality being that these scripts don't actually even work any longer.

Change-Id: I9d4f0e055a19bf662661f47ea328ec919e03ddda
2016-03-16 18:01:27 +01:00
Jenkins 3032bd0bbd Merge "Bump up undercloud VM specs" 2016-01-18 19:05:00 +00:00
Jenkins d59fc9a3dc Merge "Fix similar vm name on instack-virt-setup script." 2015-12-18 11:55:29 +00:00
Jenkins eb14b95acf Merge "Remove unused UNDERCLOUD_OS variable" 2015-11-22 19:33:24 +00:00
Jiri Stransky 67ab38fdd3 Bump up undercloud VM specs
No longer tie the default specs of undercloud VM to the overcloud VM
specs, make undercloud VM more powerful by default.

Change-Id: I53330bf7370f472edea7307883116197e3f66a63
2015-11-09 15:51:33 +01:00
Pierre Blanc 57fed7ef54 Fix similar vm name on instack-virt-setup script.
instack-virt-setup runs
"tripleo get-vm-mac $UNDERCLOUD_VM_NAME" to get the
MAC address of the instack.

If multiple vm names contain $UNDERCLOUD_VM_NAME string,
several MACs are returned.

This fix returns mac from only one virtual server.

Depends-On: I3168c2c63c2d87aa243ab819b9977ec659721026

Change-Id: I55417bff0d6f0016d12cbc3abb8906b831a2077d
2015-11-05 16:23:53 +00:00
Gael Chamoulaud bb7a70f0b6 Revive bashate tox test
- This patch fixes bashate errors.
- Bashate will ignore E006 check (Lines too long)

Change-Id: I257157a7a2cd55437e95b926ae4a7ada2a41ada5
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
2015-11-03 10:20:05 +01:00
Thomas Herve 75e7a7f835 Don't use dnsmasq lease file to retrieve address
Since libvirt 1.2.11, libvirt doesn't ask for dnsmasq to fill a lease
file, thus instack-virt-setup fails to retrieve it at the end of the
installation. Simply using arp fixes that.

Change-Id: Ie6757a7b0c224d89ebab992ef147b6fc04f774cc
Closes-Bug: #1504179
2015-10-08 17:43:15 +02:00
Jeff Peeler 424cb0eec1 Remove unused UNDERCLOUD_OS variable
This appears to be left over from when using virt-builder.

Change-Id: I67b0ca633feeed548fa85e19db1a477658a779ea
2015-10-05 17:35:39 -04:00
Gael Chamoulaud 5a1e0d8c2d Fix small typo in instack-virt-setup script
- s/distritubion/distribution

Change-Id: If24ee922340c95aeaf94c1114173e8814ea80a23
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
2015-09-21 13:28:12 +00:00
Pierre Blanc bcbf52f8d9 Customize the virt-setup log during a virtual installation
Currently the virtual deployment creates a log file called virt-setup.log
This patch customizes this name file with the name of the instack server.
So by default the log file will change from virt-setup.log to virt-setup-instack.log

Change-Id: I64762fc9209fef93c3888752ab44d5bfc4d7b6f2
2015-08-29 21:09:34 -04:00
James Slagle f5c3257407 Merge "Allow for a serial console to be attached to the instack VM." 2015-08-24 20:43:53 +02:00
James Slagle 874eb3aa83 Merge "Disable attaching extra disks by default" 2015-08-24 20:37:46 +02:00
Pierre Blanc a68251833c Add bridge name on virtual setup
Add bridge name variable on virt-setup.sh script to custom its bridge name

Change-Id: Idffade10599846f6163f258a2888a5b95d8352b3
2015-08-10 10:36:43 -04:00
James Slagle bc0afabcde Merge "Use selinux=enforcing for Overcloud image builds" 2015-07-06 21:12:43 +02:00
Mathieu Bultel df1ef382dc Fix grep command into neutron agent-list
Change-Id: I79295020bb663b248368c19ca6039ebcf946420b
2015-07-06 14:31:36 +02:00
James Slagle d8d89c99ef Use selinux=enforcing for Overcloud image builds
This will also be fixed in the unified CLI:
https://review.gerrithub.io/237539

I'm fixing it here as well just in case we have to fall back to this script for
any reason, and I don't want there to be a regression.

Change-Id: I9f1db761cf6ef5e02e79e6aed874a06d879b1e72
Related: rhbz#1233063
2015-06-24 17:11:36 -04:00
Ben Nemec 9cbf88b37c Fix passing of TESTENV_ARGS
Bash was doing some weird things with the quoting of this variable,
so we need to wrap the whole call in an eval.

Change-Id: I596f1762e0f4685458933efdbc4d3ea26981fdd4
bz: https://bugzilla.redhat.com/show_bug.cgi?id=1228419
2015-06-16 16:53:23 +00:00
James Slagle 21e49210ec Merge "Log instack-virt-setup output" 2015-06-15 22:55:55 +02:00
Ben Nemec 6faf47b65a Log instack-virt-setup output
This can be helpful for post-mortem debugging of virt-setup problems.

Change-Id: I3d76d7d7cb43594952708da25286b463ea74978f
bz: https://bugzilla.redhat.com/show_bug.cgi?id=1227936
2015-06-15 15:28:35 +02:00
Ben Nemec 5edbc7c6f1 Allow passing additional args to devtest_testenv
This should enable us to configure the environment for the split
networks testing.

Change-Id: I9d34a774e96bdc07ca424f4c9076089f7e6d7cdb
bz: https://bugzilla.redhat.com/show_bug.cgi?id=1228419
2015-06-15 15:22:55 +02:00
Lee Yarwood 773e02b374 Allow for a serial console to be attached to the instack VM.
Allows users to use `virsh console instack`.

Change-Id: I4646f64fa9e525acd1163422a5277bf79981cb51
2015-06-10 21:51:41 +02:00
Ben Nemec 89bf768f26 Deprecate local grub2 element
Upstream diskimage-builder has merged a version of the grub2 element,
so we don't need this one anymore and it conflicts with the upstream
version.  Rename it to grub2-deprecated until we can switch to the
upstream element.

Change-Id: I264d2ecfceed730ccf396e1c7338d4c670a4466a
2015-06-05 13:43:35 -05:00
Ben Nemec 6f5aabcf77 Disable attaching extra disks by default
This isn't generally necessary right now, and there was actually
a typo in the $VM_LAST_NUMBER reference that made this fail anyway.

Change-Id: Icd468769d7adca511275090265ab098367e0ffec
2015-06-04 16:18:15 -05:00
James Slagle 3d06235a5f Merge "Reduce sleep that was used to work around TFTP problems" 2015-05-27 18:43:48 +02:00
James Slagle 1318f763bc Merge "Make moving to AVAILABLE state more robust" 2015-05-27 18:43:31 +02:00
James Slagle ac5982b44a Merge "Add instack-tempest-test-overcloud" 2015-05-27 18:03:07 +02:00
Ben Nemec 902a84cea5 Merge "Fix discovery ramdisk build with RHOS" 2015-05-27 17:50:54 +02:00
Ben Nemec 50a700d00f Merge "Simplify satellite registration" 2015-05-27 17:42:31 +02:00
Dmitry Tantsur c0d8e5a745 Reduce sleep that was used to work around TFTP problems
Change-Id: I5dec611d26c98e1224fece8f1c321dfcffe36ef4
2015-05-27 15:08:46 +02:00
Dmitry Tantsur ae12f090a4 Make moving to AVAILABLE state more robust
* Move code after introspection, so that people don't report errors
  against discoverd
* Increase retry timeout to 120 seconds for all Ironic operations

Change-Id: Ie6c6af223dd6a022fdbfafe7c44bf96b6e34ecd7
2015-05-27 15:06:45 +02:00
James Slagle 60eebd0095 Fix discovery ramdisk build with RHOS
Don't build with delorean-rdo-management in the discovery ramdisk if RHOS=1

Change-Id: I2d6d920124c270a54d87807ba3bd435e0806974f
2015-05-26 16:51:41 -04:00
Ben Nemec a82f4dc363 Pythonize instack-install-undercloud
-Very incomplete testing right now
-Puts the password and stackrc files in the current user's home
 directory during the install.  Given that they now have secure
 permissions and we recommend doing that anyway, I think it's fine,
 but it is a non-trivial change in behavior.

 This was done because it's awkward to read a root-owned file from
 a Python process running as a regular user.
-Uses oslo.config instead of the bash-style answers file. A sample
 conf file created by the oslo.config generator is included (for
 now, although we may want to generate that dynamically at some
 point).  Backwards compatibility with existing answers files is
 maintained for now, but is deprecated.
-Hard-codes the image path in instack-test-overcloud to .  It's
 difficult to extract the value from the conf file in bash (unless
 they overrode the default, there's nothing for ConfigParser to read),
 and since it's just a simple sanity test script I think that's okay,
 at least for now.

Change-Id: I09270997dea7fdad2b40dfb303967ff425b55a9b
2015-05-26 14:02:18 +02:00
Dmitry Tantsur bf9e96920d Stop using RHOS variables on centos
Should fix https://bugzilla.redhat.com/show_bug.cgi?id=1224584

Change-Id: I7c72d903d4e068ea09a00835355d931497ad6c80
2015-05-25 20:42:02 +02:00
James Slagle e77c2189e5 Allow $RHOS to be overridden
This is needed to enable building images without RDO and epel.

Change-Id: I73d304d64b78376409756cef22dcc12c6d6042fb
2015-05-22 09:31:05 -07:00