Skip to content

(Maint) Consistently use the same agent fqdn#581

Merged
slippycheeze merged 1 commit intopuppetlabs:2.7.xfrom
joshcooper:maint/2.7.x/fix-fqdn-rendezvous
Mar 15, 2012
Merged

(Maint) Consistently use the same agent fqdn#581
slippycheeze merged 1 commit intopuppetlabs:2.7.xfrom
joshcooper:maint/2.7.x/fix-fqdn-rendezvous

Conversation

@joshcooper
Copy link
Copy Markdown
Contributor

Previously, the test would always fail when the rhel6-32-1 box was
reverted to its git snapshot, and the test executed. The box, for
better or worse, reports the following:

[root@rhel6-32-1 ~]# hostname -f
rhel6-32-1
[root@rhel6-32-1 ~]# facter fqdn
rhel6-32-1.puppetlabs.lan

The agent submits its report using the latter, but the test attempts to
retrieve the facts using the former:

curl -k -H \"Accept: yaml\" https://rhel6-32-1:8140/override/facts/`hostname -f`"

And that always fails:

Could not find facts rhel6-32-1

This issue does not occur when running the non-Windows acceptance tests,
because those jobs run on centos-55-386-1 and ubuntu-1004-32-1, which
don't have inconsistency in their domain as reported by facter and
hostname -f

This commit changes the test to request the facts from the master using
the same fqdn that the agent used in its report.

Previously, the test would always fail when the rhel6-32-1 box was
reverted to its `git` snapshot, and the test executed. The box, for
better or worse, reports the following:

    [root@rhel6-32-1 ~]# hostname -f
    rhel6-32-1
    [root@rhel6-32-1 ~]# facter fqdn
    rhel6-32-1.puppetlabs.lan

The agent submits its report using the latter, but the test attempts to
retrieve the facts using the former:

    curl -k -H \"Accept: yaml\" https://rhel6-32-1:8140/override/facts/`hostname -f`"

And that always fails:

    Could not find facts rhel6-32-1

This issue does not occur when running the non-Windows acceptance tests,
because those jobs run on centos-55-386-1 and ubuntu-1004-32-1, which
don't have inconsistency in their domain as reported by facter and
`hostname -f`

This commit changes the test to request the facts from the master using
the same fqdn that the agent used in its report.
slippycheeze added a commit that referenced this pull request Mar 15, 2012
(Maint) Consistently use the same agent fqdn
@slippycheeze slippycheeze merged commit 96eda07 into puppetlabs:2.7.x Mar 15, 2012
melissa pushed a commit to melissa/puppet that referenced this pull request Mar 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants