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

Task 1.1.2 - YAML Syntax #13

Open
Craconia opened this issue Oct 20, 2020 · 2 comments
Open

Task 1.1.2 - YAML Syntax #13

Craconia opened this issue Oct 20, 2020 · 2 comments

Comments

@Craconia
Copy link

Craconia commented Oct 20, 2020

Hi. Thanks for commiting the reported issues. This is one I forgot to report last time. Section 1 fails right now because of task 1.1.2.

A "colon" is needed after block and the "dashes" for the template & service tasks...and the according indentation.

CURRENTLY:

- name: "1.1.2 Ensure /tmp is configured (Scored)\n
  1.1.3 Ensure nodev option set on /tmp partition (Scored)\n
  1.1.4 Ensure nosuid option set on /tmp partition (Scored)\n
  1.1.5 Ensure noexec option set on /tmp partition (Scored)"
  block
    template:
      src: etc/systemd/system/tmp.mount.j2
      dest: /etc/systemd/system/tmp.mount
      owner: root
      group: root
      mode: 0644
    notify: restart tmp.mount

    service:
      name: tmp.mount
      state: started
      enabled: true

FIXED:

- name: "1.1.2 Ensure /tmp is configured (Scored)\n
  1.1.3 Ensure nodev option set on /tmp partition (Scored)\n
  1.1.4 Ensure nosuid option set on /tmp partition (Scored)\n
  1.1.5 Ensure noexec option set on /tmp partition (Scored)"
  block:
    - template:
        src: etc/systemd/system/tmp.mount.j2
        dest: /etc/systemd/system/tmp.mount
        owner: root
        group: root
        mode: 0644
      notify: restart tmp.mount

    - service:
        name: tmp.mount
        state: started
        enabled: true
@admlko
Copy link

admlko commented Jan 8, 2021

I can confirm the issue and the fix.

@jorritfolmer
Copy link

Here is a patch you can apply with git apply task112-patch.txt: task112-patch.txt

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

No branches or pull requests

3 participants