Skip to content
This repository has been archived by the owner on Jul 3, 2023. It is now read-only.

Use AWS IAM role #38

Merged
merged 9 commits into from
Oct 18, 2022
Merged

Use AWS IAM role #38

merged 9 commits into from
Oct 18, 2022

Conversation

atanasdinov
Copy link
Contributor

@atanasdinov atanasdinov commented Sep 1, 2022

Description

What

  • Upgrading Go to version 1.18
  • Upgrading klauspost/compress to latest version improving the Snappy compression
  • Migrating from rlmcpherson/s3gof3r to aws/aws-sdk-go
  • Migrating from gopkg.in/mgo.v2 to go.mongodb.org/mongo-driver
  • Updating documentation
  • Minor lint / construction fixes

Why

JIRA Ticket

Anything, in particular, you'd like to highlight to reviewers

Scope and particulars of this PR (Please tick all that apply)

  • Tech hygiene (dependency updating & other tech debt)
  • Bug fix
  • Feature
  • Documentation
  • Breaking change
  • Minor change (e.g. fixing a typo, adding config)

This Pull Request follows the rules described in our Pull Requests Guide

@atanasdinov atanasdinov force-pushed the feature/CPH-282_migrate-to-iamroles branch from 7a83611 to e97d102 Compare September 1, 2022 13:51
@atanasdinov atanasdinov force-pushed the feature/CPH-282_migrate-to-iamroles branch 2 times, most recently from bebd043 to 34a9ec5 Compare September 5, 2022 16:07
@atanasdinov atanasdinov force-pushed the feature/CPH-282_migrate-to-iamroles branch from 34a9ec5 to 76c4300 Compare September 7, 2022 06:04
@atanasdinov atanasdinov force-pushed the feature/CPH-282_migrate-to-iamroles branch from 2ab0ad8 to 9853da5 Compare September 13, 2022 11:17
@atanasdinov atanasdinov force-pushed the feature/CPH-282_migrate-to-iamroles branch from a2a8f22 to 35729a8 Compare September 13, 2022 15:06
@atanasdinov atanasdinov marked this pull request as ready for review October 12, 2022 12:41
@atanasdinov atanasdinov requested a review from a team as a code owner October 12, 2022 12:41
Copy link
Contributor

@ivanruski ivanruski left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

return err
}

func newPipe(op operation) (io.ReadCloser, io.WriteCloser) {
Copy link
Contributor

Choose a reason for hiding this comment

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

nitpick (non-blocking): newPipe is only used in backupService.go. We can move it there.

@dbelev dbelev merged commit 6b14306 into master Oct 18, 2022
@dbelev dbelev deleted the feature/CPH-282_migrate-to-iamroles branch October 18, 2022 06:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants