Skip to content
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

debian bullseye support #89

Merged
merged 4 commits into from
Oct 5, 2021
Merged

debian bullseye support #89

merged 4 commits into from
Oct 5, 2021

Conversation

pmauduit
Copy link
Member

@pmauduit pmauduit commented Sep 2, 2021

See #88.

  • fixes the ansible-galaxy version number for the elasticsearch role
  • migrates the vagrant box to 'debian/bullseye64', the playbook has been adapted to remain compatible with buster, though
  • adding a serverspec testsuite for vagrant, documenting how to use it

tests: ansible-playbook went well when vagrant provision on both versions, but the serverspec testsuite reveals some caveats:

  • there is an non critical issue at tomcat-proxycas's startup, and the SSL/TLS connector is unavailable (so nothing is actually listening onto the 8443 tcp port)
  • as no smtp is provided by default, the datafeeder is unable to run, so the test on the df port is also failing

tested by removing the roles & relaunching:
`ansible-galaxy install -r requirements.yaml`
Tests: tested against a debian/buster64 & a debian/bullseye vagrant box.
The playbook does not fail "per se", but the serverspec testsuite
reveals that:

* there is an issue with the SSL connector onto the tomcat-proxycas
  instance.
* Vagrant boxes do not provide a default SMTP, and the Datafeeder
  backend refuses to launch if it cannot reach a SMTP server (on
  localhost:25 by default).
@pmauduit
Copy link
Member Author

pmauduit commented Sep 3, 2021

There are still some parts of the playbook's roles which are relying on being under a debian/buster env, this one still needs some rework

@landryb
Copy link
Member

landryb commented Sep 6, 2021

i'm fine with this being merged as soon as you consider it ready :)

@landryb landryb merged commit 70c941b into master Oct 5, 2021
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