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

Format LVM volumes asynchronously #181

Merged
merged 4 commits into from Jul 27, 2018

Conversation

Zempashi
Copy link

Fixes: #173

ChangeLog.rst Outdated
@@ -13,6 +13,10 @@ upgrading a MetalK8s 0.1 cluster to MetalK8s 0.2 (:ghissue:`147`)
storage related actions. Please see :ref:`upgrade_from_MetalK8s_before_0.2.0`
(:ghissue:`153`)

Bugs fixed
Copy link
Contributor

Choose a reason for hiding this comment

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

Not really a bug...

Copy link
Author

@Zempashi Zempashi Jul 23, 2018

Choose a reason for hiding this comment

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

I wondered a long time about this... So it's a feature added with a ghissue ?

Copy link
Contributor

Choose a reason for hiding this comment

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

There are only 'Issues' on Github (and 'Notes' in a 'Project'), so yeah. It's a bit of a misnomer, but we've been using 'Issues' to track features for quite a while already...

Copy link

@ballot-scality ballot-scality left a comment

Choose a reason for hiding this comment

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

The value of async / poll and retries are not very clear to me.

Also I would very like to have some value in the defaults/main.yml of this role to have a way to modify these values.

@NicolasT NicolasT changed the title Format lvm lv asynchronously Format LVM volumes asynchronously Jul 24, 2018
@NicolasT
Copy link
Contributor

Can this be retargetted to development/1.0?

@Zempashi Zempashi changed the base branch from development/0.2 to development/1.0 July 25, 2018 10:08
Copy link
Contributor

@NicolasT NicolasT left a comment

Choose a reason for hiding this comment

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

Needs some ChangeLog fixes.

ChangeLog.rst Outdated
@@ -38,6 +38,10 @@ upgrading a MetalK8s 0.1 cluster to MetalK8s 0.2 (:ghissue:`147`)
storage related actions. Please see :ref:`upgrade_from_MetalK8s_before_0.2.0`
(:ghissue:`153`)

Features added
Copy link
Contributor

Choose a reason for hiding this comment

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

This should go in the 1.0.0 section

ChangeLog.rst Outdated
@@ -38,6 +38,10 @@ upgrading a MetalK8s 0.1 cluster to MetalK8s 0.2 (:ghissue:`147`)
storage related actions. Please see :ref:`upgrade_from_MetalK8s_before_0.2.0`
(:ghissue:`153`)

Features added
--------------
:ghissue:`173` - Format pv in asynchronous mode (:ghpull:`181`)
Copy link
Contributor

Choose a reason for hiding this comment

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

:ghpull:`181` - format `PersistentVolumes` asynchronously (:ghissue:`173`)

NicolasT
NicolasT previously approved these changes Jul 26, 2018
@NicolasT NicolasT added kind:enhancement New feature or request topic:storage Issues related to storage labels Jul 27, 2018
@NicolasT NicolasT added this to In progress in Sprint to MetalK8s 1.0.0 via automation Jul 27, 2018
@NicolasT NicolasT added this to the MetalK8s 1.0.0 milestone Jul 27, 2018
@NicolasT NicolasT moved this from In progress to Reviewer approved in Sprint to MetalK8s 1.0.0 Jul 27, 2018
@Zempashi Zempashi merged commit a4bf9ee into development/1.0 Jul 27, 2018
Sprint to MetalK8s 1.0.0 automation moved this from Reviewer approved to Done Jul 27, 2018
@Zempashi Zempashi deleted the asynchronous_lv_formating branch July 27, 2018 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind:enhancement New feature or request topic:storage Issues related to storage
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants