Skip to content

Conversation

@ricardomaraschini
Copy link
Contributor

in order to capture problems with the embedded license we need to use a version of the embedded cluster binary with an embedded kots license.

@ricardomaraschini ricardomaraschini force-pushed the use-binary-with-license-for-testing branch from f167eac to d47b65a Compare November 8, 2023 10:59
@ricardomaraschini ricardomaraschini marked this pull request as draft November 8, 2023 11:01
@ricardomaraschini ricardomaraschini force-pushed the use-binary-with-license-for-testing branch from d47b65a to 445f764 Compare November 8, 2023 11:02
in order to capture problems with the embedded license we need to use a
version of the embedded cluster binary with an embedded kots license.
@ricardomaraschini ricardomaraschini force-pushed the use-binary-with-license-for-testing branch from 445f764 to 21b415f Compare November 8, 2023 13:20
Comment on lines 22 to 24
expect "Enter a new Admin Console password:" { send "password\r" }
expect "Confirm password:" { send "password\r" }
expect "Admin Console is ready!"
expect "Confirm password:" { send "password\r" }
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we really have to type the password three times?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My mistake. Removed.

@ricardomaraschini ricardomaraschini marked this pull request as ready for review November 8, 2023 19:43
@ricardomaraschini ricardomaraschini merged commit 781f6b9 into main Nov 8, 2023
@ricardomaraschini ricardomaraschini deleted the use-binary-with-license-for-testing branch November 8, 2023 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants