Make PrmContainerSetEACL fields public for client and pool #148

aarifullin · 2023-08-18T15:10:29Z

aarifullin commented

2023-08-18 15:10:29 +00:00

Signed-off-by: Airat Arifullin a.arifullin@yadro.com

Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>

aarifullin added the

refactoring

label 2023-08-18 15:10:29 +00:00

aarifullin added 1 commit 2023-08-18 15:10:35 +00:00

22978303f8 [#121 ] clientt: Make PrmContainerSetEACL fields public

Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>

acid-ant approved these changes 2023-08-20 17:04:23 +00:00

acid-ant approved these changes 2023-08-20 17:05:39 +00:00

dkirillov reviewed 2023-08-21 06:22:53 +00:00

pool/pool.go Outdated

													
				@ -570,2 +568,2 @@

					if !prm.waitParamsSet {

						prm.waitParams.setDefaults()

					if prm.WaitParams == nil {

						prm.WaitParams = defaultWaitParams()

dkirillov commented

2023-08-21 06:22:53 +00:00

It seems now we can drop WaitParams.setDefaults because it isn't used anymore

It seems now we can drop `WaitParams.setDefaults` because it isn't used anymore

aarifullin commented

2023-08-21 07:33:58 +00:00

Poster

Done

dkirillov marked this conversation as resolved

aarifullin force-pushed feature/121-prm_set_cont_eacl from da44a0bab3 to 342524159a

2023-08-21 07:33:46 +00:00

Compare

dkirillov approved these changes 2023-08-21 07:55:05 +00:00

fyrchik approved these changes 2023-08-21 08:42:00 +00:00

fyrchik merged commit 342524159a into master

2023-08-21 08:42:19 +00:00

dkirillov referenced this pull request from TrueCloudLab/frostfs-s3-gw

2023-08-21 09:23:51 +00:00

[#180] Fix linter issues #191

Sign in to join this conversation.

No reviewers

No Label

No Milestone

No Assignees

4 Participants

Notifications

Due Date

The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: TrueCloudLab/frostfs-sdk-go#148