lzap
August 20, 2015, 12:40pm
1
Hello,
I will be presenting latest changes of Discovery image, plugin and smart
theforeman:master ← lzap:tui
opened 02:04PM - 07 Aug 15 UTC
The code is done, _feel free to review and test now_.
See the README change for… instructions how this is supposed to work.
**Pull requests to merge with this**:
- [x] Discovery plugin: https://github.com/theforeman/foreman_discovery/pull/210
- [x] Smart-proxy discovery image (new) plugin: https://github.com/theforeman/smart_proxy_discovery_image/pull/1
- [ ] Smart-proxy discovery "proxy": https://github.com/theforeman/smart_proxy_discovery/pull/8 (not blocking)
- [ ] Documentation: https://github.com/theforeman/theforeman.org/pull/421 (not blocking)
Instructions:
Build image with this patch and make sure you provide explicit version to it `3.0.0`. This is important, if you build with latest tag `2.1.1` it will not work. Give it any release number you want like 1 or 99.
```
./build-livecd fdi-centos7.ks 99 3.0.0 && sudo ./build-livecd-root 3.0.0
```
Version can be checked on the TUI interface, in the `/usr/share/fdi/VERSION` file or uploaded facts.
Then checkout foreman develop with foreman_discovery with the patch above. Checkout smart-proxy with discovery "proxy" plugin enabled and with the patch above (note the difference between discovery image and discovery proxy plugins).
The last remaining bit is to install smart_proxy_discovery into the image. You need to do it via ssh since there is no RPM package yet: `gem build smart_proxy_discovery_image.gemspec && sshpass -p redhat scp smart_proxy_discovery_image-1.0.0.gem root@192.168.122.68: && sshpass -p redhat ssh root@192.168.122.68 "gem install smart_proxy_discovery_image-1.0.0.gem && echo 'gem \"smart_proxy_discovery_image\"' > /usr/share/foreman-proxy/bundler.d/discovery_image.rb && systemctl restart foreman-proxy && tail -f /tmp/proxy.log"`
Known issues (TODO):
- [x] better semi-automatic mode (currently it is all or nothing)
- [ ] document how to modify kernel command line options ISO file
- [x] better taxonomy support for discovered hosts (https://github.com/theforeman/foreman_discovery/pull/211)
- [x] better screen texts and buttons
- [x] do IP address CIDR validation on the network form
- [x] move the README block in the official docs
- [x] test with DHCP (but no PXE)
- [x] test without DHCP with multiple NICs
- [x] added extra "append" fact into the template to workaround above
When testing this, do all three scenarios:
- DHCP & PXE (traditional discovery, interrupt with a key)
- DHCP only
- no DHCP
Join our YouTube stream or Hangouts if you plan to actively participate.
Hangout link:
Youtube link (stream&recording):
···
--
Later,
Lukas #lzap Zapletal
