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

SYS-607 update gitlab helm chart to 16.3.3; tweak ansible for virtualbox #132

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

instantlinux
Copy link
Owner

Summary of Changes

Housekeeping update:

  • Gitlab helm chart now supports version 16.3.3, which retired the logging_level setting for gitaly. (There is still 20x as much log output from gitlab container as there should be -- almost none of it useful -- and nobody at GitLab wants to tackle this problem after years of complaints.)
  • Replaced deprecated apt_key resource with get_url in ansible fileserver role: I dunno why the folks at ansible didn't just fix apt_key to do the right thing for debian-based distros

Why is this change being made?

Support latest versions to comply with current security best practices.

How was this tested? How can the reviewer verify your testing?

Brought up gitlab, verified functionality; cleared out virtualbox repo settings and confirmed the new resource definition works properly.

Completion checklist

  • The pull request is linked to all related issues
  • This change has unit test coverage
  • Documentation has been updated
  • Dependencies have been updated and verified

@instantlinux instantlinux added the dependencies Pull requests that update a dependency file label Sep 14, 2023
@instantlinux instantlinux merged commit a46f1f3 into main Sep 14, 2023
1 check passed
@instantlinux instantlinux deleted the SYS-607_update3 branch September 14, 2023 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant