Skip to content
Snippets Groups Projects

Run instances with IMDSv2

Merged Thiago Presa requested to merge tpresa-run-instances-imdsv2 into master
All threads resolved!

:warning:NOTE THAT THIS FORK IS MAINTAINED FOR CRITICAL BUG FIXES AFFECTING RUNNING COSTS ONLY. NO OTHER CONTRIBUTIONS WILL BE ACCEPTED. :warning:

What critical bug this MR is fixing?

Closes #15 (closed)

How does this change help reduce cost of usage? What scale of cost reduction is it?

In what scenarios is this change usable with GitLab Runner's docker+machine executor?

Edited by Thiago Presa

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • @tpresa Back to you with my comments :slight_smile:

  • Tomasz Maczukin assigned to @tpresa and unassigned @tmaczukin

    assigned to @tpresa and unassigned @tmaczukin

  • Thiago Presa added 1 commit

    added 1 commit

    Compare with previous version

  • Thiago Presa resolved all threads

    resolved all threads

  • Thiago Presa added 1 commit

    added 1 commit

    Compare with previous version

  • Thiago Presa added 5 commits

    added 5 commits

    Compare with previous version

  • Thiago Presa marked this merge request as draft from 6de1ec3b

    marked this merge request as draft from 6de1ec3b

  • Thiago Presa marked this merge request as ready

    marked this merge request as ready

  • Thiago Presa assigned to @tmaczukin and unassigned @tpresa

    assigned to @tmaczukin and unassigned @tpresa

  • Thanks for the review! PTAL.

  • Tomasz Maczukin assigned to @tpresa and unassigned @tmaczukin

    assigned to @tpresa and unassigned @tmaczukin

  • Thiago Presa added 1 commit

    added 1 commit

    Compare with previous version

  • Thiago Presa assigned to @tmaczukin and unassigned @tpresa

    assigned to @tmaczukin and unassigned @tpresa

  • Thiago Presa resolved all threads

    resolved all threads

  • Tomasz Maczukin approved this merge request

    approved this merge request

  • Tomasz Maczukin started a merge train

    started a merge train

  • Thanks for handling this @tpresa! Thanks for the merge @tmaczukin. :)

  • Joshua Lambert mentioned in issue #93

    mentioned in issue #93

  • Please register or sign in to reply
    Loading