summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Replace cryptic error with better oneFlavio Castelli2017-05-26
| | | | | | | | | | When the remote source was not found the check for its size failed because the size was empty, causing the to integer conversion to fail. The new code ensures the remote file exists before trying to convert its size. Signed-off-by: Flavio Castelli <fcastelli@suse.com>
* Merge pull request #107 from eamonnotoole/remote-ignition-temp-file-upstreamFlavio Castelli2017-05-26
|\ | | | | Write Ignition file as a volume in a libvirt storage pool
| * Write Ignition file as a volume in a libvirt storage poolEamonn O'Toole2017-03-23
| | | | | | | | | | | | | | | | | | | | | | This avoids the problem where the Ignition file is remote to the host on which the libvirt domain is being created. We've added a "libvirt_ignition" resource which manages the Ignition file in the libvirt volume - creates it, and destroys it. The "coreos_ignition" field in the libvirt_domain definition must point to the Id of a "libvirt_ignition" resource. The code is modelled on that used for CloudInit.
* | Merge pull request #116 from dmacvicar/if_modified_sinceFlavio Castelli2017-05-26
|\ \ | | | | | | Use If-Modified-Since for downloading images
| * | Use If-Modified-Since for downloading imagesAlvaro Saurin2017-03-29
| | |
* | | Fix travis: require latest stable release of GoFlavio Castelli2017-05-03
| | | | | | | | | | | | | | | The terraform code is taking advantage of features available only inside of go 1.8
* | | Update travis - do not use govendFlavio Castelli2017-05-03
| | | | | | | | | | | | | | | This is no longer needed now that all the deps are stored inside of the git repository.
* | | Vendor dependencies with vndrFlavio Castelli2017-05-03
|/ / | | | | | | This fixes issue #123
* | Code refactoring: rename structFlavio Castelli2017-03-29
| |
* | cloud-init: rework user-data handlingFlavio Castelli2017-03-29
| | | | | | | | | | | | | | | | Merge the user data specified by explicit terraform directives into the raw data provided by the user. The raw data has priority over the values specified using older directives. Signed-off-by: Flavio Castelli <fcastelli@suse.com>
* | DocsAlvaro Saurin2017-03-29
| |
* | user_data argument for cloud-initAlvaro Saurin2017-03-29
|/
* Add scsi controller option and wwn to virtual disksEamonn O'Toole2017-03-22
| | | | | | | | | | | | | | | | | | | We've added the facility to define a disk bus type of scsi to the specification of a disk in the domain definition. If the disk is a scsi disk, a random wwn is generated unless a wwn is provided for that disk. The disk stanza now looks as follows: disk { volume_id = "${libvirt_volume.mydisk.id}" scsi = "yes" wwn = "05abcd123456789a" } Having "scsi" present with any value will specify a scsi bus for the disk. If "wwn" is present for a scsci disk then the value of "wwn" is used for the disk wwn, otherwise a random value is generated.
* Merge pull request #103 from monstermunchkin/graphics-deviceFlavio Castelli2017-03-15
|\ | | | | create domain without graphics device
| * Add test for graphicsThomas Hipp2017-03-15
| | | | | | | | Signed-off-by: Thomas Hipp <thipp@suse.de>
| * Exclude `graphics` block on s390x and ppc64Thomas Hipp2017-03-15
| | | | | | | | | | | | | | Since the `graphics` and `video` devices cause problems on s390x and ppc64, they are ignored when creating a domain. Signed-off-by: Thomas Hipp <thipp@suse.de>
* | Increase code coverage by running TF acceptance testsEamonn O'Toole2017-03-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This has required a number of changes, first of all to .travis.yml to allow the TF acceptance tests to run: 1. To avoid the need to build libvirt, we've added the Ubuntu "cloud-archive:mitaka" repo which contains more recent versions of libvirt than those that ship with 14.04. This also cuts a couple of minutes off the travis run-time. 2. We need to define and start a default storage pool, since one doesn't exist by default on 14.04. 3. We need to add the travis user to the libvirtd group, and then run both tests with "sg" to set the libvirtd group without requiring a log-out/log-in 4. We need to set a couple of global environment variables for the travis run: - TF_ACC to true to ensure that TF acceptance tests are run - LIBVIRT_DEFAULT_URI needs to be set, we set it to qemu://system. - We've added a new environment variable that can be used to set the domain type for test purposes. We need to set the domain type to "qemu" for travis CI. "test" doesn't implement all of the libvirt functions that are used, and "kvm" isn't yet available in travis. 5. We have to add "-v" to the test and goveralls command to ensure that TF acceptance tests are run. In addition, we had to make the following change to resource_libvirt_domain_test.go - The "br0" bridge doesn't exist by default, so instead we use the default network in resource_libvirt_domain_test.go With these changes the test coverage goes up to 44% as measured by goveralls. Because we're using qemu emulation, the tests take about 2 mins to run on travis. We might not need to run both the tests and goveralls; one might do.
* | Some code coverage for utils.goFlavio Castelli2017-03-14
| |
* | Test coverage for network_defFlavio Castelli2017-03-14
| |
* | Code coverage of qemu_agentFlavio Castelli2017-03-14
| | | | | | | | | | Small refactoring needed to have unit test coverage of qemu_agent.go
* | Test coverage for pool syncFlavio Castelli2017-03-14
|/
* Merge pull request #114 from MalloZup/issue-templateFlavio Castelli2017-03-13
|\ | | | | Adding issue template
| * Adding issue templatedario2017-03-13
|/
* Merge pull request #106 from MalloZup/improveDocInstallFlavio Castelli2017-03-13
|\ | | | | simplify the install documentation
| * Remove deprecated dotfileDario Maiocchi2017-03-02
| |
| * simplify the install documentationDario Maiocchi2017-03-02
| | | | | | | | | | Setting $GOPATH and installing go should be a prereq of go-devs. Additionally go get -u does already the clone in src
* | Merge pull request #112 from MalloZup/more-verboseFlavio Castelli2017-03-13
|\ \ | | | | | | Be more verbose/precise by debugging failures.
| * | Be more verbose/precise by debugging failures.dmaiocchi2017-03-10
|/ / | | | | | | | | We have a race condition here, see issue 88 and printing same string for two differents error in code is not the best solution for debug.
* | gofmtAlvaro Saurin2017-03-06
| |
* | Improve code coverageFlavio Castelli2017-03-03
| | | | | | | | | | | | | | Improve code coverage results by writing some unit tests. These are really the low hanging fruits, more should come later. Signed-off-by: Flavio Castelli <fcastelli@suse.com>
* | Update README.mdFlavio Castelli2017-03-02
| | | | | | Add coverall badge
* | Try to enable Go code coverage on coverallsFlavio Castelli2017-03-02
|/
* Merge pull request #101 from rsokolkov/fix-docs-for-networkAlvaro2017-02-20
|\ | | | | [Docs] Clearer description for "addresses" param
| * [Docs] Clearer description for "addresses" paramRoman Sokolkov2017-02-17
| |
* | Merge pull request #102 from deasmi/bridge_fixAlvaro2017-02-20
|\ \ | |/ |/| Bridge fix
| * Merge remote-tracking branch 'origin' into bridge_fixDean Smith2017-02-17
| |\ | |/ |/|
* | Merge pull request #95 from eamonnotoole/hpe/networking_dnsFlavio Castelli2017-02-17
|\ \ | | | | | | Add DNS forwarder blocks to network resource definition
| * | Add DNS forwarder blocks to network resource definitionEamonn O'Toole2017-02-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There can be one or more of these optional blocks. Each block looks as follows: dns_forwarder { address = "my address" domain = "my domain" } One or both of "address" and "domain" can be specified in each block.
* | | Merge pull request #96 from eamonnotoole/hpe/ignitionFlavio Castelli2017-02-17
|\ \ \ | |/ / |/| | Add support for CoreOS ignition and console blocks
| * | Ignition support: first phaseEamonn O'Toole2017-02-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A few changes here: 1. CoreOS Ignition support: A CoreOS Ignition file can be specified for a domain using the "coreos_ignition" parameter. Alternatively the "coreos_ignition" parameter can be set equal to a Terraform ignition object. If the latter, the ignition object is written to a file in /tmp whose name is a hash of the object itself. This file-name is stored in libvirt domain metadata, and is removed when the domain is destroyed. This feature requires the emission of qemu:commandline XML XML and also the setting of the XML name-space to "http://libvirt.org/schemas/domain/qemu/1.0" 2. "graphics" block: A "graphics" block can be specified in a domain definition. We've added this because we've found that for some builds of qemu the default "spice" emulator doesn't work and results in failure to boot the VMs. 3. "console" block: One or more "console" blocks can be specified in a domain definition. Note the description in domain.html.markdown, and the information in https://libvirt.org/formatdomain.html#elementsConsole Added a test for coreos_ignition in resource_libvirt_domain_test.go
| * | Add support for CoreOS ignition and console blocksEamonn O'Toole2017-02-08
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A few changes here: 1. CoreOS Ignition support: A CoreOS Ignition file can be specified for a domain using the "coreos_ignition" parameter. This requires the emission of additional XML and also the setting of the XML name-space to "http://libvirt.org/schemas/domain/qemu/1.0" 2. "graphics" block: A "graphics" block can be specified in a domain definition. We've added this because we've found that for some builds of qemu the default "spice" emulator doesn't work and results in failure to boot the VMs. 3. "console" block: One or more "console" blocks can be specified in a domain definition. Note the description in domain.html.markdown, and the description in https://libvirt.org/formatdomain.html#elementsConsole
| * Fix for bridged networks.Dean Smith2017-02-17
|/ | | | | | | Need to make domain attribute optional, as not used for bridge Disabling forwarding for bridged networks Ensure that parsing existing network doesn't fail due to lack of domain attribute
* Merge pull request #94 from monstermunchkin/travis-build-issueFlavio Castelli2017-01-26
|\ | | | | travis: fix build issue
| * travis: fix build issueThomas Hipp2017-01-26
|/ | | | | | | | | When building with Go 1.6, the build fails due to the missing "context" package. Using Go 1.7 solves this issue. Beside that, Terraform requires Go 1.7. Signed-off-by: Thomas Hipp <thipp@suse.de>
* remove "experimental"Duncan Mac-Vicar P2016-12-16
|
* Merge pull request #91 from monstermunchkin/masterFlavio Castelli2016-12-14
|\ | | | | vendor: bump terraform to v0.8.0
| * vendor: bump terraform to v0.8.0Thomas Hipp2016-12-14
|/ | | | Signed-off-by: Thomas Hipp <thipp@suse.de>
* Merge pull request #90 from moio/masterFlavio Castelli2016-12-14
|\ | | | | Documentation: add output attributes for domain and volume
| * Documentation: add output attributes for domain and volumeSilvio Moioli2016-12-13
|/
* Build against latest stable release of terraformFlavio Castelli2016-11-25
|
generated by cgit v1.2.3 (git 2.25.4) at 2024-07-07 06:32:20 +0000