Skip to content

Enable krb5kdc-fixture, kerberos tests mount urandom for kdc container#41710

Merged
bizybot merged 11 commits intoelastic:masterfrom
bizybot:fix/40624
Jun 13, 2019
Merged

Enable krb5kdc-fixture, kerberos tests mount urandom for kdc container#41710
bizybot merged 11 commits intoelastic:masterfrom
bizybot:fix/40624

Conversation

@bizybot
Copy link
Copy Markdown
Contributor

@bizybot bizybot commented May 1, 2019

Infra has fixed https://github.com/elastic/infra/issues/10462 by installing haveged on CI workers.
This commit enables the disabled fixture and tests, and mounts
/dev/urandom for the container so there is enough
entropy required for kdc.

Closes #40624,#40678

Infra has fixed elastic#10462 by installing `haveged` on CI workers.
This commit enables the disabled fixture and tests, and mounts
`/dev/urandom` for the container so there is enough
entropy required for kdc.

Closes elastic#40624
@bizybot bizybot added >test Issues or PRs that are addressing/adding tests :Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) v8.0.0 v6.7.3 v7.2.0 labels May 1, 2019
@bizybot bizybot requested a review from alpar-t May 1, 2019 11:33
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-security

@bizybot
Copy link
Copy Markdown
Contributor Author

bizybot commented May 2, 2019

Connection error failed HDFS tests, seems one off as I could not see any test failures reported.
@elasticmachine run elasticsearch-ci/1

command: "bash /fixture/src/main/resources/provision/peppa.sh"
volumes:
- ./build/shared/peppa:/fixture/build
- /dev/urandom:/dev/random
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Maybe worth to add a comment as to why this is necessary, it resembles a typo on first glance.

Copy link
Copy Markdown
Contributor

@alpar-t alpar-t left a comment

Choose a reason for hiding this comment

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

thanks @bizybot !

@jakelandis jakelandis added v6.8.1 and removed v6.8.0 labels May 19, 2019
@bizybot bizybot added v7.3.0 and removed v7.2.0 labels Jun 12, 2019
@bizybot bizybot merged commit 4757870 into elastic:master Jun 13, 2019
bizybot added a commit to bizybot/elasticsearch that referenced this pull request Jun 13, 2019
elastic#41710)

Infra has fixed elastic#10462 by installing `haveged` on CI workers.
This commit enables the disabled fixture and tests, and mounts
`/dev/urandom` for the container so there is enough
entropy required for kdc.
Note: hdfs-repository tests have been disabled, will raise a separate issue for it.

Closes elastic#40624 Closes elastic#40678
bizybot added a commit to bizybot/elasticsearch that referenced this pull request Jun 13, 2019
elastic#41710)

Infra has fixed elastic#10462 by installing `haveged` on CI workers.
This commit enables the disabled fixture and tests, and mounts
`/dev/urandom` for the container so there is enough
entropy required for kdc.
Note: hdfs-repository tests have been disabled, will raise a separate issue for it.

Closes elastic#40624 Closes elastic#40678
bizybot added a commit to bizybot/elasticsearch that referenced this pull request Jun 13, 2019
elastic#41710)

Infra has fixed elastic#10462 by installing `haveged` on CI workers.
This commit enables the disabled fixture and tests, and mounts
`/dev/urandom` for the container so there is enough
entropy required for kdc.
Note: hdfs-repository tests have been disabled, will raise a separate issue for it.

Closes elastic#40624 Closes elastic#40678
bizybot added a commit to bizybot/elasticsearch that referenced this pull request Jun 13, 2019
elastic#41710)

Infra has fixed elastic#10462 by installing `haveged` on CI workers.
This commit enables the disabled fixture and tests, and mounts
`/dev/urandom` for the container so there is enough
entropy required for kdc.
Note: hdfs-repository tests have been disabled, will raise a separate issue for it.

Closes elastic#40624 Closes elastic#40678
bizybot added a commit that referenced this pull request Jun 13, 2019
#41710) (#43179)

Infra has fixed #10462 by installing `haveged` on CI workers.
This commit enables the disabled fixture and tests, and mounts
`/dev/urandom` for the container so there is enough
entropy required for kdc.
Note: hdfs-repository tests have been disabled, will raise a separate issue for it.

Closes #40624 Closes #40678
bizybot added a commit that referenced this pull request Jun 13, 2019
#41710) (#43178)

Infra has fixed #10462 by installing `haveged` on CI workers.
This commit enables the disabled fixture and tests, and mounts
`/dev/urandom` for the container so there is enough
entropy required for kdc.
Note: hdfs-repository tests have been disabled, will raise a separate issue for it.

Closes #40624 Closes #40678
bizybot added a commit that referenced this pull request Jun 13, 2019
#41710) (#43180)

Infra has fixed #10462 by installing `haveged` on CI workers.
This commit enables the disabled fixture and tests, and mounts
`/dev/urandom` for the container so there is enough
entropy required for kdc.
Note: hdfs-repository tests have been disabled, will raise a separate issue for it.

Closes #40624 Closes #40678
bizybot added a commit that referenced this pull request Jun 13, 2019
#41710) (#43181)

Infra has fixed #10462 by installing `haveged` on CI workers.
This commit enables the disabled fixture and tests, and mounts
`/dev/urandom` for the container so there is enough
entropy required for kdc.
Note: hdfs-repository tests have been disabled, will raise a separate issue for it.

Closes #40624 Closes #40678
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Security/Authentication Logging in, Usernames/passwords, Realms (Native/LDAP/AD/SAML/PKI/etc) >test Issues or PRs that are addressing/adding tests v6.8.1 v7.3.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

krb5kdc-fixture intermittently fails to start

5 participants