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

5.0.0 / 2023-11-03 #30

Merged
merged 1 commit into from
Nov 29, 2023
Merged

5.0.0 / 2023-11-03 #30

merged 1 commit into from
Nov 29, 2023

Conversation

halostatue
Copy link
Member

@halostatue halostatue commented Oct 12, 2023

  • Upgraded to Python 3.12

  • Upgraded to Ansible 8.5.0

  • Renamed the run script to kineticcafe-ansible and fixed several issues:

    • Bash 4 or later is required for associative array support. Ensured that
      this would be respected on macOS by using /usr/bin/env bash instead of
      /bin/bash.

    • Updated the script to use the current version of the image.

    • Fixed various issues with file and directory mounts. Many more mountable
      files (--become-password-file, etc.) are supported than
      --vault-password-file. Note that not all possible file parameters are
      supported, such as --module-paths or --extra-vars @file. Pull requests
      for supporting these would be considered.

    • Fixed an overzealous application of --ask-vault-password, even for
      commands that could not use it.

    • Changed the sh subcommand to execute bash and added bash as a known
      subcommand.

    • Added support for deriving the entry point from basename $0.

  • Updated the Docker image to use HEREDOC.

    • Added less, nano, and vim-nox packages.
  • Added an install script to install kineticcafe-ansible and optional
    symlinks.

@halostatue halostatue force-pushed the update-and-add-setup-script branch 2 times, most recently from 2f8ef92 to f48b387 Compare October 30, 2023 17:03
@halostatue halostatue linked an issue Oct 30, 2023 that may be closed by this pull request
@halostatue halostatue marked this pull request as ready for review October 30, 2023 17:13
@halostatue halostatue self-assigned this Oct 30, 2023
ohnit
ohnit previously approved these changes Oct 30, 2023
@halostatue halostatue changed the title 4.1.0 / 2023-10-12 4.1.0 / 2023-10-31 Oct 30, 2023
@halostatue halostatue enabled auto-merge (rebase) October 30, 2023 18:20
Copy link

@mikestok mikestok left a comment

Choose a reason for hiding this comment

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

I don't understand enough to provide useful comments.

- Upgraded to Python 3.12

- Upgraded to Ansible 8.5.0

- Renamed the `run` script to `kineticcafe-ansible` and fixed several issues:

  - Bash 4 or later is required for associative array support. Ensured that
    this would be respected on macOS by using `/usr/bin/env bash` instead of
    `/bin/bash`.

  - Updated the script to use the current version of the image.

  - Fixed various issues with file and directory mounts. Many more mountable
    files (`--become-password-file`, etc.) are supported than
    `--vault-password-file`. Note that not _all_ possible file parameters are
    supported, such as `--module-paths` or `--extra-vars @file`. Pull requests
    for supporting these would be considered.

  - Fixed an overzealous application of `--ask-vault-password`, even for
    commands that could not use it.

  - Changed the `sh` subcommand to execute `bash` and added `bash` as a known
    subcommand.

  - Added support for deriving the entry point from `basename $0`.

- Updated the Docker image to use HEREDOC.

  - Added `less`, `nano`, and `vim-nox` packages.

- Added an `install` script to install `kineticcafe-ansible` and optional
  symlinks.
@halostatue halostatue changed the title 4.1.0 / 2023-10-31 5.0.0 / 2023-11-03 Nov 2, 2023
@halostatue halostatue merged commit 0b23928 into main Nov 29, 2023
1 check passed
@halostatue halostatue deleted the update-and-add-setup-script branch November 29, 2023 15:07
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.

README references the wrong docker image
3 participants