Skip to content

C2DEVEL-XXXX: add legacy-xfs option#19

Merged
ancient07 merged 4 commits intoC2Devel:masterfrom
ancient07:fix-xfs-error
Jan 13, 2025
Merged

C2DEVEL-XXXX: add legacy-xfs option#19
ancient07 merged 4 commits intoC2Devel:masterfrom
ancient07:fix-xfs-error

Conversation

@ancient07
Copy link

Is this a bug fix or adding new feature?

What is this PR about? / Why do we need it?

What testing is done?

@ancient07 ancient07 requested a review from astroline October 18, 2024 14:30
fs.StringVar(&s.UserAgentExtra, "user-agent-extra", "", "Extra string appended to user agent.")
fs.BoolVar(&s.Batching, "batching", false, "To enable batching of API calls. This is especially helpful for improving performance in workloads that are sensitive to EC2 rate limits.")
fs.DurationVar(&s.ModifyVolumeRequestHandlerTimeout, "modify-volume-request-handler-timeout", driver.DefaultModifyVolumeRequestHandlerTimeout, "Timeout for the window in which volume modification calls must be received in order for them to coalesce into a single volume modification call to AWS. This must be lower than the csi-resizer and volumemodifier timeouts")
f.BoolVar(&s.LegacyXFSProgs, "legacy-xfs", false, "Warning: This option will be removed in a future version of EBS CSI Driver. Formats XFS volumes with `bigtime=0,inobtcount=0,reflink=0`, so that they can be mounted onto nodes with linux kernel ≤ v5.4. Volumes formatted with this option may experience issues after 2038, and will be unable to use some XFS features (for example, reflinks).")

Choose a reason for hiding this comment

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

fs.BoolVar должно же быть

Copy link
Author

Choose a reason for hiding this comment

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

спасибо, исправлю

@ancient07 ancient07 merged commit 9b28965 into C2Devel:master Jan 13, 2025
2 checks passed
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.

3 participants