Project

General

Profile

Activity

From 07/08/2014 to 08/06/2014

08/06/2014

10:01 AM Feature #6542 (Closed): Ship IPA user script with Smart Proxy
Applied in changeset commit:c54e833e964ef0fb8f7c17fb9ad685dc43c22403. Anonymous
10:01 AM Refactor #6522 (Closed): Move ForemanRequest class out of the chef plugin
Applied in changeset commit:6108539b6e15aed6ad1c97a6b44ef0a54ccbd20e. Anonymous
10:01 AM Refactor #6743 (Closed): Common code for manage_subnet? in ISC and Native MS
Applied in changeset commit:3925ae4deefec15b02486666569bbd74c93915d3. Aaron Stone
10:01 AM Refactor #6744 (Closed): Avoid nested arrays and extra array copies in the ISC DHCP config parser
Applied in changeset commit:776461f89d34dbce7016bf3e944ccc8a67e22e73. Aaron Stone
10:01 AM Bug #6745 (Closed): Clean up the params input to the DHCP::Server.addRecord method
Applied in changeset commit:dd9d295bc439d467bbfc455b28b63d37176dea21. Aaron Stone
09:09 AM Bug #6745 (Ready For Testing): Clean up the params input to the DHCP::Server.addRecord method
Dominic Cleal
10:01 AM Refactor #6742 (Closed): Add a unit test suite for the DHCP API
Applied in changeset commit:247c5b5186e6e63303ee314eaca93afd3dbdd3d7. Aaron Stone
09:11 AM Revision c54e833e: fixes #6542 - ship freeipa users script with the smart proxy
Stephen Benjamin
09:10 AM Revision 6108539b: fixes #6522 - move request code out of chef module
The patch also moves foreman_ssl_ca, foreman_ssl_cert and
foreman_ssl_key config options from Chef plugin config to g...
Martin Milata
09:10 AM Revision 3925ae4d: Fixes #6743 - Common code for manage_subnet? in ISC and Native MS
Aaron Stone
09:10 AM Revision 776461f8: Fixes #6744 - Avoid nested arrays and extra array copies in the ISC DHCP config parser
Aaron Stone
09:10 AM Revision dd9d295b: Fixes #6745 - Clean up the params input to the DHCP::Server.addRecord method
This fixes the ability to match and ignore pre-existing identical DHCP records. Aaron Stone
09:10 AM Revision 247c5b51: Fixes #6742 - Add a unit test suite for the DHCP API
This is built around the ISC DHCPD provider, the goal is testing the API layer. Aaron Stone
08:58 AM Bug #6956: foreman-proxy requires rubygem-rack => 1.1.0
Likely the same for debs too. Dominic Cleal
08:57 AM Bug #6956 (Closed): foreman-proxy requires rubygem-rack => 1.1.0
When trying to start the foreman-proxy service, it would start and then crash with no errors. This was due to the ha... Leah Fisher
08:45 AM Bug #6945: smart-proxy doesn't find the autosign.conf if not at "#{puppetdir}/autosign.conf"
Brian Witt wrote:
> I thought the puppet.conf location was set via :puppet_conf:
Yeah, puppet_conf will override ...
Dominic Cleal
08:41 AM Bug #6945: smart-proxy doesn't find the autosign.conf if not at "#{puppetdir}/autosign.conf"
I thought the puppet.conf location was set via :puppet_conf: Brian Witt
03:43 AM Bug #6945: smart-proxy doesn't find the autosign.conf if not at "#{puppetdir}/autosign.conf"
Brian Witt wrote:
> Our puppetdir in this case is set to /etc/puppet. Is there anything else that uses puppetdir?
...
Dominic Cleal
06:26 AM Bug #6908 (Closed): Chef and Puppet request test failures
Dominic Cleal
06:24 AM Bug #6908 (Resolved): Chef and Puppet request test failures
commit 37c1113f5c1b7c896238f8e0f764241c7bd1cdb7 Anonymous
06:22 AM Revision 2781dc7d: Merge pull request #194 from domcleal/6908-request-tests
fixes #6908 - update request tests for latest Chef release Dmitri Dolguikh
04:40 AM Feature #4171 (Duplicate): Smart proxy DHCP plugin does not honor exclude ranges
#1280 also tracks this issue. Dominic Cleal

08/05/2014

03:22 PM Bug #6945: smart-proxy doesn't find the autosign.conf if not at "#{puppetdir}/autosign.conf"
Our puppetdir in this case is set to /etc/puppet. Is there anything else that uses puppetdir? Brian Witt
03:19 PM Bug #6945 (Closed): smart-proxy doesn't find the autosign.conf if not at "#{puppetdir}/autosign.conf"
modules/puppetca/puppetca.rb hardocodes the autosign.conf file to
def autosign_file
"#{puppetdir}/autos...
Brian Witt
08:32 AM Bug #6589 (Ready For Testing): Trusted host list seems to be ignored
The Foreman Bot
05:12 AM Refactor #6743 (Ready For Testing): Common code for manage_subnet? in ISC and Native MS
Dominic Cleal
05:11 AM Refactor #6744 (Ready For Testing): Avoid nested arrays and extra array copies in the ISC DHCP config parser
Dominic Cleal
04:53 AM Refactor #6742 (Ready For Testing): Add a unit test suite for the DHCP API
Dominic Cleal

08/04/2014

11:15 AM Bug #6916 (Resolved): MS DHCP Timeout
Hi,
just updated to Foreman 1.5.2 today as I already had issues with 1.5.1 deploying hosts. Sometimes that host cr...
Oliver Weinmann
07:14 AM Bug #6908 (Ready For Testing): Chef and Puppet request test failures
The Foreman Bot
07:13 AM Bug #6908 (Closed): Chef and Puppet request test failures
Proxy tests are failing with a difference between "/" and "//" in the Chef request URLs. Looks like it's from the re... Dominic Cleal
07:14 AM Revision 37c1113f: fixes #6908 - update request tests for latest Chef release
Dominic Cleal

08/01/2014

04:43 PM Bug #6893 (Rejected): Class import tries to update the same classes over and over again.
I've long witnessed this with 1.2.x and seen it again since upgrading to 1.5.2 yesterday. Joshua Hoblitt

07/30/2014

11:01 AM Bug #5675: DNS Proxy Issues
Mike McRill wrote:
> Changing dns_key to false in settings.yaml resolved the issue, thank you!
As we just discove...
Dominic Cleal

07/29/2014

12:05 PM Bug #6819 (Closed): proxy should raise an exception when it can't acquire a lock on a temp file
Ran into this issue after a disk filled up on a DHCP proxy and had been cleaned up.
To reproduce, touch /tmp/fore...
Greg Petras

07/28/2014

05:01 AM Bug #6790 (Closed): ci_reporter 2.0 gem breaks CI
Applied in changeset commit:0e616f1b0be11f51f19d08e407fa987b3ea5abc7. Aaron Stone
04:01 AM Bug #6790 (Ready For Testing): ci_reporter 2.0 gem breaks CI
The Foreman Bot
03:59 AM Bug #6790 (Closed): ci_reporter 2.0 gem breaks CI
Example output:
our Gemfile lists the gem gettext (~> 2.0) more than once.
You should probably keep only one of t...
Dominic Cleal
04:40 AM Feature #6788 (Ready For Testing): Add a puppet class filter to the scanner to filter puppet class imports based on directory or name
Dominic Cleal
04:39 AM Revision 0e616f1b: Fixes #6790: Restrict ci_reporter gem to less than 2.0.0 to fix CI.
Aaron Stone

07/27/2014

08:55 AM Feature #6788: Add a puppet class filter to the scanner to filter puppet class imports based on directory or name
The patch speeded up imports on our puppet env from 7+ minutes to about 10 seconds due to the amount of environments ... Nils Domrose
08:54 AM Feature #6788: Add a puppet class filter to the scanner to filter puppet class imports based on directory or name
https://github.com/theforeman/smart-proxy/pull/191 Nils Domrose
06:43 AM Feature #6788 (Rejected): Add a puppet class filter to the scanner to filter puppet class imports based on directory or name
Several puppet deployments follow the module, profile(aka service), role pattern. Using this pattern, only roles will... Nils Domrose

07/24/2014

12:01 PM Revision dc635bfe: Fixes #6772: added support for testing of plugins
Dmitri Dolguikh

07/23/2014

10:48 AM Bug #6718 (Need more information): Unused IP request with virsh errors with: unable to retrive virsh info: Must define a name
I've just reproduced, but I am not sure if this is a bug. Just add the missing info there, virsh adds these records c... Lukas Zapletal
09:01 AM Feature #4699 (Closed): Support Puppet environmentpath and environment config files
Applied in changeset commit:16db7a9c74abfba21b65d9f0e9a05096c3c7f8fb. Dominic Cleal
08:33 AM Revision 16db7a9c: fixes #4699 - support Puppet environments API for directory envs
Dominic Cleal
06:44 AM Feature #4630 (Resolved): Enhance Smart Proxy to detect pulp
This was implemented as a plugin (success!) over here: https://github.com/theforeman/smart-proxy-pulp, published as s... Dominic Cleal
02:13 AM Bug #6745 (Closed): Clean up the params input to the DHCP::Server.addRecord method
Clean up the params input to the DHCP::Server.addRecord method
Actual:
Add a DHCP record to local DHCP config fil...
Aaron Stone
02:11 AM Refactor #6744 (Closed): Avoid nested arrays and extra array copies in the ISC DHCP config parser
Avoid nested arrays and extra array copies in the ISC DHCP config parser Aaron Stone
02:11 AM Refactor #6743 (Closed): Common code for manage_subnet? in ISC and Native MS
Common code for manage_subnet? in ISC and Native MS Aaron Stone
02:10 AM Refactor #6742 (Closed): Add a unit test suite for the DHCP API
Add a unit test suite for the DHCP API. Aaron Stone

07/22/2014

10:25 AM Bug #6718: Unused IP request with virsh errors with: unable to retrive virsh info: Must define a name
Maybe it's these.. I have some DHCP reservations with no hostnames - maybe that's from before #4661? In which case I... Dominic Cleal
10:00 AM Bug #6718: Unused IP request with virsh errors with: unable to retrive virsh info: Must define a name
How to reproduce? WFM... Lukas Zapletal
08:48 AM Bug #6718 (Rejected): Unused IP request with virsh errors with: unable to retrive virsh info: Must define a name
Since #6377, unused IP requests to the proxy when using DHCP + virsh provider now result in:... Dominic Cleal
04:01 AM Bug #6689 (Closed): "ERROR -- : Invalid MAC address:" using the "Build PXE Default" button
Applied in changeset commit:bdc3b51e9aaf02ddc4e1582e16547a0fae269fd1. Trey Dockendorf
03:13 AM Revision bdc3b51e: Fixes #6689 - Allow nil mac address for TFTP API calls that do not require a MAC
Trey Dockendorf

07/21/2014

02:37 PM Bug #6689 (Ready For Testing): "ERROR -- : Invalid MAC address:" using the "Build PXE Default" button
The Foreman Bot
02:32 PM Bug #6689: "ERROR -- : Invalid MAC address:" using the "Build PXE Default" button
I see where #6341 went wrong. PR #189 has been uploaded to fix the regression. Trey Dockendorf
04:34 AM Bug #6689: "ERROR -- : Invalid MAC address:" using the "Build PXE Default" button
Looks like 1.5 allowed a blank MAC address via the API, while #6341 changed it so it didn't. Dominic Cleal
11:01 AM Bug #6700 (Closed): Proxy::Puppet::Runner incorrectly checked $? for success
Applied in changeset commit:2cba6063e429fa6aed365e8950200207c2fb8a5c. Aaron Stone
10:45 AM Bug #6700 (Closed): Proxy::Puppet::Runner incorrectly checked $? for success
"The popen test was looking at $?, but it wasn't set by the method under test. Testing Ruby's popen method isn't terr... Dominic Cleal
10:46 AM Revision 2cba6063: fixes #6700 - tests for Proxy::Puppet::Runner shell methods
Aaron Stone

07/18/2014

09:56 AM Bug #6689 (Closed): "ERROR -- : Invalid MAC address:" using the "Build PXE Default" button
I did a clean install with foreman 1.6 nightly today and when I use the "Build PXE Default" button, I get an error:
...
Jorick Astrego

07/17/2014

03:18 PM Feature #6677 (Resolved): Autosign entry additions should require authentication
Using the smart proxy API, I can create an autosign entry by POSTing to http://smartproxy.example.com/puppet/ca/autos... Michael Messmore
08:01 AM Bug #6661 (Closed): GET /dhcp is always 404
Applied in changeset commit:99339284ea7c5fd967ae014f584fe4e3bfdcd364. Anonymous
06:54 AM Bug #6661 (Ready For Testing): GET /dhcp is always 404
The Foreman Bot
06:44 AM Bug #6661 (Assigned): GET /dhcp is always 404
Anonymous
04:06 AM Bug #6661 (Closed): GET /dhcp is always 404
when trying to fetch proxy subnets via a API request (GET /dhcp/) I get the following:... Ohad Levy
07:59 AM Revision 99339284: fixes #6661: trailing slash is not required in urls
Dmitri Dolguikh

07/15/2014

04:01 AM Bug #6588 (Closed): Tests for default settings are taking local config files into account
Applied in changeset commit:a52a4f69a265270ba78ca58437f332ead138520a. Dominic Cleal

07/14/2014

03:43 PM Bug #5571: Puppet "each" function makes Foreman not show a class
Upon further investigation, it seems to be a problem with determining which parser to use when it reads from puppet.c... Jason Knudsen
01:54 PM Bug #5571: Puppet "each" function makes Foreman not show a class
Oh and yes, I have parser = future in the main and master sections of puppet.conf Jason Knudsen
01:53 PM Bug #5571: Puppet "each" function makes Foreman not show a class
I'm also seeing an issue whereby the foreman_proxy fails to import classes if they have 'future parser' code within.
...
Jason Knudsen

07/11/2014

12:36 PM Bug #6588 (Ready For Testing): Tests for default settings are taking local config files into account
The Foreman Bot
12:36 PM Bug #6588 (Assigned): Tests for default settings are taking local config files into account
Dominic Cleal
12:23 PM Bug #6588 (Ready For Testing): Tests for default settings are taking local config files into account
The Foreman Bot
09:32 AM Bug #6588 (Closed): Tests for default settings are taking local config files into account
I have some settings which appear to interfere with unit tests for default settings:... Dominic Cleal
12:12 PM Revision a52a4f69: fixes #6588 - don't load user settings when testing default settings
Dominic Cleal
09:40 AM Bug #6589 (Closed): Trusted host list seems to be ignored
The trusted host list does not seem to be used (migrate_settings.rb only moves the configuration value around).
<p...
Martin Milata
09:15 AM Feature #4699: Support Puppet environmentpath and environment config files
No worries, I'm very glad somebody with a non-standard SSL setup has tried it too, thanks for the feedback! Dominic Cleal
09:07 AM Feature #4699: Support Puppet environmentpath and environment config files
It did thanks thought I had tried that and many other options besides. And then rather embarrasingly I left a typo in... Paul Seymour
09:00 AM Feature #4699: Support Puppet environmentpath and environment config files
Setting puppet_url should do the trick, but it has to be a full URL, not a hostname: @:puppet_url: https://puppet3.do... Dominic Cleal
07:54 AM Feature #4699: Support Puppet environmentpath and environment config files
Hi - I have applied the patch and now I am getting this error:-
E, [2014-07-11T10:50:32.833458 #7140] ERROR -- : F...
Paul Seymour
06:01 AM Bug #6585 (Closed): Fix command exit code handling under Ruby 1.9+
Applied in changeset commit:bbafbc039a20baa6b3b62622afc9a145b4fabc22. Aaron Stone
05:39 AM Bug #6585 (Closed): Fix command exit code handling under Ruby 1.9+
The exit codes from executed commands aren't retrieved properly, this can be fixed for Ruby 1.9+. Dominic Cleal
06:01 AM Bug #6584 (Closed): Replace Kernel#open with File.open
Applied in changeset commit:a220e9a9bd3697ced5bf89b6fac52496f6796b51. Aaron Stone
05:38 AM Bug #6584 (Closed): Replace Kernel#open with File.open
Some uses of Kernel.open instead of File.open, so it only opens files rather than URLs etc. Dominic Cleal
06:01 AM Refactor #6583 (Closed): Use Enumerable#find instead of each where possible
Applied in changeset commit:aa478589395d782f058798b77e88ec2ad55157ef. Aaron Stone
05:37 AM Refactor #6583 (Closed): Use Enumerable#find instead of each where possible
Some untidy code exists which could use Enumerable#find instead of returning nil. Dominic Cleal
06:01 AM Bug #6412 (Closed): Trying to remove a DHCP record fails due to an invalid subnet check
Applied in changeset smart-proxy:commit:57ea7e82f68395c61ab0515ea331423c9f4ac5ea. Anonymous
06:01 AM Bug #6396 (Closed): an error removing a DHCP record (record doesn't exist) stops the delete process for a host
Applied in changeset commit:67bfd9a09741726cf7fa7c330b897b372a903d31. Anonymous
05:42 AM Revision bbafbc03: fixes #6585 - Update Proxy::Util::CommandTask to handle Ruby 1.8 and 1.9+
In this update, use 'true' and 'false' to test $? return codes 0 and 1, respectively.
In Ruby 1.8.x, the return code ...
Aaron Stone
05:38 AM Revision a220e9a9: fixes #6584 - Replace Kernel#open with File.open
Aaron Stone
05:36 AM Revision aa478589: fixes #6583 - Use Enumerable#find instead of each where possible
Aaron Stone
05:29 AM Revision 57ea7e82: Fixes #6412: Trying to remove a DHCP record fails due to an invalid subnet check
David Swift
05:29 AM Revision 67bfd9a0: Fixes #6396: an error removing a DHCP record (record doesn't exist) stops the delete process for a host
David Swift

07/10/2014

03:08 AM Bug #6557: Puppet module needs debug logging
It's simply that the Puppet module in the proxy hasn't got a single line of debug, though it indeed should! Dominic Cleal

07/09/2014

05:10 PM Bug #6557 (New): Puppet module needs debug logging
I set the loglevel in a smartproxy that only does "puppet" to "debug", and the logfile output did not display anythin... Simon Mügge
08:08 AM Feature #6542: Ship IPA user script with Smart Proxy
PR https://github.com/theforeman/smart-proxy/pull/184 Stephen Benjamin
07:59 AM Feature #6542 (Closed): Ship IPA user script with Smart Proxy
Ship the script here with the Smart Proxy - http://projects.theforeman.org/projects/foreman/wiki/IPASmartProxyUser Stephen Benjamin
05:01 AM Bug #6275 (Closed): migrate_settings.yml doesn't return 1 on migrated config file
Applied in changeset commit:14fc2cedbe4e360a905a45089848be36af670ab2. Dominic Cleal
04:47 AM Revision 14fc2ced: fixes #6275 - idempotence needs clone of original data and to handle *_port
Dominic Cleal

07/08/2014

10:54 AM Bug #6526 (Resolved): Registering the smart-proxy service on windows fails.
Hi,
I can't get the smart-proxy service registered on a Windows 2008 R2 server.
I receive the following error w...
Oliver Weinmann
08:27 AM Refactor #6522 (Ready For Testing): Move ForemanRequest class out of the chef plugin
https://github.com/theforeman/smart-proxy/pull/182 Dominic Cleal
08:08 AM Refactor #6522 (Closed): Move ForemanRequest class out of the chef plugin
It does not currently contain any chef-specific code and it's easier for other plugins to use if it's in the core. Martin Milata
03:08 AM Bug #6396: an error removing a DHCP record (record doesn't exist) stops the delete process for a host
Don't worry, it wasn't that important. Thanks for checking David. Dominic Cleal
 

Also available in: Atom