Skip to content
This repository has been archived by the owner on Nov 6, 2024. It is now read-only.

New provider: OpenStack #170

Merged
merged 10 commits into from
Nov 5, 2018
Merged

New provider: OpenStack #170

merged 10 commits into from
Nov 5, 2018

Conversation

dnsmichi
Copy link
Contributor

Thanks @martbhell for the initial implementation.

@martbhell
Copy link
Contributor

Cool stuff!
I guess this is still a work in progress? Would be nice with some documentation how to use this.

I needed to set:

  • OS_SSH_USER
  • OS_SSH_PRIVATE_KEY_PATH
  • OS_KEYPAIR_NAME
  • OS_IMAGE
  • OS_FLAVOR

and also modify the Vagrantfile (or create a security group called Vagrant)

I still got the "Unsupported OS" on my CentOS-7 image - because https://github.com/Icinga/icinga-vagrant/blob/master/.puppet/scripts/shell_provisioner_pre.sh#L19 conditional was not cool - made a PR for that: #171>
After that the vagrant and puppet completed and I could login to an icinga web interface :)

os.password = "#{ENV['OS_PASSWORD']}"
os.flavor = "#{ENV['OS_FLAVOR']}" # TODO: config
os.image = "#{ENV['OS_IMAGE']}" # TODO: config
os.project_name = "#{ENV['OS_PROJECT_NAME']}" # TODO: config
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OS_PROJECT_NAME should not be a config/todo either, it comes from the openrc file too.

Copy link
Contributor

@martbhell martbhell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some documentation would be sweet :)

Michael Friedrich added 4 commits November 4, 2018 15:25
…visioner

Needed for more OpenStack options, and helps reduce cluttering among different
boxes all sharing the same describing code.

This also silences the Puppet deprecation warnings, thanks @lazyfrosch
@dnsmichi
Copy link
Contributor Author

dnsmichi commented Nov 4, 2018

Docs was still in my local working copy, it is now pushed.

@dnsmichi
Copy link
Contributor Author

dnsmichi commented Nov 4, 2018

I've also refactored the Vagrantfile files for each project into generic helper functions, which allows for easier modifications.

@dnsmichi
Copy link
Contributor Author

dnsmichi commented Nov 4, 2018

The config TODOs have been removed, this is left for the user to set.

@dnsmichi
Copy link
Contributor Author

dnsmichi commented Nov 4, 2018

I've tested this locally with Parallels, remotely with Openstack for standalone, influxdb and elastic. It is important to open up the security groups and exposed ports to allow intercommunication via external floating ip. This affects Graphite, InfluxDB, Elastic, etc. and is especially noted in the documentation.

The docs are moved to the doc/ btw, as it doesn't fit into the general README (which is for basic users, not advanced OpenStack users).

@dnsmichi
Copy link
Contributor Author

dnsmichi commented Nov 4, 2018

Need to fix a parameter problem for Elasticsearch http proxy, fix is coming.

This also removes the Nginx server IP binding, which isn't possible with
OpenStack and the external routed IP address ("floating IP").
@dnsmichi
Copy link
Contributor Author

dnsmichi commented Nov 5, 2018

Fixed it, needed to move the mapped port range for Kibana/ES to 5602/9202.

@dnsmichi dnsmichi merged commit 802889a into master Nov 5, 2018
@dnsmichi dnsmichi deleted the feature/provider-openstack branch November 5, 2018 10:01
dnsmichi pushed a commit that referenced this pull request Aug 16, 2019
e0bf8cfe Merge pull request #179 from olevole/master
8cd1f6e0 fix spec: proper data_dir value for FreeBSD
4333ff9e properly aligned
b37bdb0d repo and package method are the same for FreeBSD
e1aeca52 fix typo
e4aa1eff repo: added case for FreeBSD OS family
d661e22c fix typo
b9e566ad Merge branch 'master' of github.com:olevole/puppet-grafana
4277076e Merge branch 'master' of github.com:olevole/puppet-grafana
b299a124 Merge pull request #180 from voxpupuli/modulesync
e6a50cc9 modulesync 2.8.0
e66e9999 Merge branch 'master' of github.com:olevole/puppet-grafana
cad5c06e Add FreeBSD platform support
b2c5b7e2 Add FreeBSD platform support
69d9cae6 Merge pull request #176 from voxpupuli/afisher_update_dependencies
8c752fee allow puppetlabs/apt 7.x
01eb3f57 Allow puppet/archive 4.x and puppetlabs/stdlib 6.x
fa9bce3d Merge pull request #172 from alexconrey/master
5c47fe5d Fix rubocop issues (#9)
ee7252df fix issue with grafana_folder idempotency apply (#8)
5f08df2f Merge pull request #171 from voxpupuli/modulesync
3011aba0 Update README.md (#7)
f7580a3b fix rubocop warnings (#6)
f86d7c30 Update grafana dashboard resource (#5)
d6a938a4 Rewrite grafana_cli unit test
cd52596e modulesync 2.7.0
d5067d36 Merge pull request #170 from alexconrey/master
2adb0966 update acceptance tests for grafana_folder (#4)
3f6d08c5 Update README.md/create acceptance tests (#3)
e1817961 rubocop fixes (#2)
29c1bfa4 Corrected invalid database config example (#169)
99254e3a Implement grafana folder resource (#1)
ab1c6f58 Use data in modules instead of params.pp (#167)
4c57681f Merge pull request #166 from dhoppe/puppet3_syntax
822a6fd5 Remove Puppet 3 specific syntax
78c361bc Mark passwords as sensitive (#165)
f4c4d9ba Merge pull request #161 from dhoppe/base_url
1b093c7a Merge pull request #164 from voxpupuli/modulesync
411ab48a Merge branch 'master' into base_url
3f98b51c Merge pull request #160 from dhoppe/real_package_source
20c26773 Update to latest available version and make sure the repo configs are unique (#163)
0a84497f modulesync 2.6.0
d5a535ef Fix value of variables base_url and real_archive_source
72ba05ae Fix value of variable real_package_source

git-subtree-dir: .puppet/modules/grafana
git-subtree-split: e0bf8cfe9ac06852fbb423e247e18e765d9d4a8b
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants