-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fedora 20 Beaker tests failing #32
Comments
I'm still seeing this after a Vagrant update.
|
I'm still seeing F20 Beaker tests fail in the way described above:
After a
I am able to manually recreate the
|
* Without this, wouldn't be provisioned via RPM or tarball * Relevant to #32 and #63 Signed-off-by: Daniel Farrell <[email protected]>
Making this Issue specific to F20, will handle F21 Beaker test failures in #63 (they are different failures). Update: After fixing that^^ F21 fail, F21 started hitting this failure |
Note that the Fedora 20 Puppet Vagrant VM provided by vagrant-opendaylight seems to work correctly. This issue is likely Beaker-specific, not a problem with the Puppet mod itself.
|
I mentioned previously that I was able to replicate the I'm also able to replicate it on via vagrant-opendaylight on an F20 VM provisioned by this Puppet module's RPM install method.
ODL does seem to be up and working.
|
Again, this this is the relevant part of the Beaker trace:
Digging through
These docs from the top of that file are somewhat informative:
It seems that we either need to support that |
I'm seeing this
Makes sense, as the
|
I noticed that parallel to our
|
Given these relevant defaults in the
I would expect our F20 system to end up using this option, as it's a more specific match than the
And it seems that the only way Puppet would execute
Maybe somehow the Beaker tests are not getting the correct Facter facts for the F20 VM? Looking at a
Again, those could trip either the |
Seeing the same expected values on a Beaker F20 Puppet RPM VM:
That VM failed the Beaker tests that were run when it was created with the same
This seems to imply that the more specific rules for choosing a default Puppet |
Grepping for default Puppet
|
These unit tests should cover the Puppet behavior (choosing more specific default OSs matches) described above.
Note that I found them in Puppet's On the failed F20 Beaker RPM box described above:
|
This commit fixed the issue I'm seeing. Note that it's in master, but not in the 3.7.4 Puppet release.
Unfortunately, 3.7.4 is the latest Puppet release. |
More git-fu to show that the relevant commit is in master but not 3.7.4:
The Here, I check out the 3.7.4 release tag and show that it was cut on Jan 26th 2015 while the commit we need was merged on January 30th 2015.
|
I labeled this as |
The fix we need was included when the Puppet 3.7.5 release was cut. Hasn't been packaged for Fedora yet:
|
Tuns out they use a different Yum repo. Once I pointed Yum at yum.puppetlabs.com I got the 3.7.5 update. Direct link to RPM.
|
All tests passing after 1f7d340. |
Both the Fedora 20 and Fedora 21 Baker tests are failing with similar (I think the same) errors. CentOS 7 is working fine.
The text was updated successfully, but these errors were encountered: