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

add molecule github actions #95

Merged
merged 35 commits into from
Jan 3, 2024
Merged

add molecule github actions #95

merged 35 commits into from
Jan 3, 2024

Conversation

lanefu
Copy link
Contributor

@lanefu lanefu commented Jul 9, 2020

No description provided.

@lanefu lanefu changed the title add molecule github actions WIP add molecule github actions Jul 9, 2020
lanefu and others added 12 commits July 8, 2020 22:24
* Update to Nomad 0.11.3 (#96)

Addresses #84

* Fix booleans in template (#94)

nomad expects booleans in lower-case. this fixes several variables

* Update nomad to 0.12.0 (#98)

Update to nomad 0.12.0 #97

* Update nomad to v0.12.1 (#104)

Addresses #103

* Improve Vault Support for Namespace, Tokens (#102)

This adds support for Vault namespaces (from Vault Enterprise).
Additionally, this ensures that a vault token is not installed on client
nodes. Per the nomad documentation, this token only needs be installed
on servers, and tokens will be delegated appropriately to client nodes.

Addresses #101

* add parameter deployment_gc_threshold (#99)

Co-authored-by: beechesII <christopher.grau@t-systems.com>

* Feature/nomad autopilot (#100)

* add configuration for Nomad autopilot Stanza

* correct variable names

Co-authored-by: beechesII <christopher.grau@t-systems.com>

* Reference requirement for unzip

In response to #89 ...  unzip is a dependency of ansible unarchive module.

* fix cgroup-bin transition package for newer distro releases (#108)

* Fix After and Wants to start after networking (#110)

According to https://www.freedesktop.org/wiki/Software/systemd/NetworkTarget/ this is the correct way to start a service only after the network has come all the way up.
According to https://www.freedesktop.org/software/systemd/man/systemd.special.html#basic.target "systemd automatically adds dependency of the type After= for this target unit to all services (except for those with DefaultDependencies=no)", therefore having it here is redundant.

* Added configuation for podman and general plugin setups. (#111)

* Added configuation for podman and general plugin setups.

* Added storage usiing the same macro.

* Removed the new hosts storage since the function existed already.

* Corrected installation order.

* CI test

* CI test

* fix action

* fix action

* CI test

* CI bug wa

* CI bug wa

* File permissions

* File permissions

* Add centos7

* File permissions

* Fix syntax

* Use sh

* Disable idempotence molecule test

* Cleaning

* fix lint

* Fix tests

* cleanup in lists of systems

* Comment all except Ubuntu-18

* Fix tests

* Add Ubuntu-20

* Set ansible-lint warnings

* Set ansible-lint warnings

* Set ansible-lint exceptions

* Temp. disable lint

* Edit scenario

* Try different image

* Edit scenario

* add molecule github actions

* add molecle tasks

* add lint forgiveness

* lint tweaks

* fix ansible lint

* remove custom yaml lint

* copied yamllint config from consul rolee

* remove document start linting

* test remove idempotence test

* comment spacing

* exclude 301, ansible lint

* use new lint syntax

* rollback base

* just 1 scenario while debugging

* fixes fro yamllint

* CI test

* CI test

* fix action

* fix action

* CI test

* CI bug wa

* CI bug wa

* File permissions

* File permissions

* Add centos7

* File permissions

* Fix syntax

* Use sh

* Disable idempotence molecule test

* Cleaning

* fix lint

* Fix tests

* cleanup in lists of systems

* Comment all except Ubuntu-18

* Fix tests

* Add Ubuntu-20

* Set ansible-lint warnings

* Set ansible-lint warnings

* Set ansible-lint exceptions

* Temp. disable lint

* Edit scenario

* Try different image

* Edit scenario

* Edit scenario

* CI test

* Clean the mess

* Clean the mess

* Rmove temporary CI rule

* CI test

* Rmove temporary CI rule

Co-authored-by: adawalli <adam.wallis@gmail.com>
Co-authored-by: Sebastian Gumprich <rndmh3ro@users.noreply.github.com>
Co-authored-by: Christopher Grau <66788631+beechesII@users.noreply.github.com>
Co-authored-by: beechesII <christopher.grau@t-systems.com>
Co-authored-by: lanefu <lanefu@users.noreply.github.com>
Co-authored-by: John Adams <john@hexb.it>
Co-authored-by: jebas <jeff.l.baskin@gmail.com>
Co-authored-by: Michal Muransky <michal.muransky@pan-net.eu>
Co-authored-by: Lane Jennison <lane@lane-fu.com>
Co-authored-by: Bas Meijer <bas.meijer@me.com>
@beechesII beechesII marked this pull request as draft December 21, 2023 13:57
@beechesII beechesII changed the title WIP add molecule github actions P add molecule github actions Jan 3, 2024
@beechesII beechesII changed the title P add molecule github actions add molecule github actions Jan 3, 2024
@beechesII
Copy link
Collaborator

beechesII commented Jan 3, 2024

@rndmh3ro PR ready for review. I fixed the molecule tests. debian-12 and ubuntu-23.04 will be fixed by #163.

I also added a action for ansible-lint. I will fix this issues in a another PR.

FYI @lanefu

@beechesII beechesII marked this pull request as ready for review January 3, 2024 11:30
Copy link
Collaborator

@rndmh3ro rndmh3ro left a comment

Choose a reason for hiding this comment

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

Awesome work!

I only have one minor change, otherwise this can be merged.

tasks/main.yml Outdated Show resolved Hide resolved
Co-authored-by: Sebastian Gumprich <rndmh3ro@users.noreply.github.com>
@beechesII beechesII merged commit 82c3714 into master Jan 3, 2024
11 of 14 checks passed
@beechesII beechesII deleted the ci-gh-actions branch January 3, 2024 12:56
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.

5 participants