Project

General

Profile

Actions

Bug #37903

closed

Ansible password is not extracted correctly in the Windows default provisioning template

Added by Nadja Heitmann 2 months ago. Updated 13 days ago.

Status:
Closed
Priority:
Normal
Category:
Unattended installations
Target version:

Description

In the Windows default provision the password value is passed like this
<Value>![CDATA[<%= host_param('ansible_ssh_pass') %>]]</Value>

which as a result looks like this

<Value>![CDATA[start123!]]</Value>

but should be

<Value><![CDATA[start123!]]></Value>

such that it can be parsed as 'start123!'

Actions #1

Updated by The Foreman Bot 2 months ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/10344 added
Actions #2

Updated by Ewoud Kohl van Wijngaarden about 2 months ago

  • Target version set to 3.11.4
  • Fixed in Releases deleted (3.11.3)

Please don't set the Fixed in Release field when opening an issue. If you want it in a specific release, please use the Target version field.

Actions #3

Updated by The Foreman Bot about 2 months ago

  • Fixed in Releases 3.13.0 added
Actions #4

Updated by Nadja Heitmann about 2 months ago

I think I could not unset it. I had the problem with another ticket as well.

Actions #5

Updated by Nadja Heitmann about 2 months ago

  • Status changed from Ready For Testing to Closed
Actions #6

Updated by Ewoud Kohl van Wijngaarden about 2 months ago

  • Category set to Unattended installations

Is this something you want cherry picked to 3.11 & 3.12? If so, you're welcome to open cherry pick PRs yourself.

Actions #7

Updated by Ewoud Kohl van Wijngaarden about 2 months ago

Oh, and in most browsers you can use ctrl+click to unselect values in multi-select fields (and shift+click to add additional values).

Actions #8

Updated by Nadja Heitmann about 2 months ago

Thanks! I'll keep that in mind for the next time!

Actions #9

Updated by The Foreman Bot about 2 months ago

  • Pull request https://github.com/theforeman/foreman/pull/10348 added
Actions #10

Updated by Patrick Creech about 2 months ago

  • Target version changed from 3.11.4 to 3.12.1
Actions #11

Updated by The Foreman Bot 20 days ago

  • Assignee set to Ewoud Kohl van Wijngaarden
  • Pull request https://github.com/theforeman/foreman/pull/10379 added
Actions #12

Updated by The Foreman Bot 19 days ago

  • Fixed in Releases 3.12.1 added
Actions #13

Updated by The Foreman Bot 13 days ago

  • Fixed in Releases 3.12.2 added
Actions

Also available in: Atom PDF