Skip to content

Conversation

@Sh4d1
Copy link
Contributor

@Sh4d1 Sh4d1 commented Mar 19, 2020

Signed-off-by: Patrik Cyvoct pcyvoct@scaleway.com

Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Sh4d1 and others added 4 commits March 19, 2020 10:59
Co-Authored-By: Rémy Léone <remy.leone@gmail.com>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Sh4d1 and others added 3 commits March 19, 2020 12:23
Co-Authored-By: Loïc Bourgois <loic@bourgois.pro>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
loicbourgois
loicbourgois previously approved these changes Mar 19, 2020
Copy link
Contributor

@loicbourgois loicbourgois left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Copy link
Contributor

@QuentinBrosse QuentinBrosse left a comment

Choose a reason for hiding this comment

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

Our README will become too big with all these details IMO.. :/

Maybe we should move this advancer configs in a dedicated file or simply provide image in docker hub right now?

@Sh4d1
Copy link
Contributor Author

Sh4d1 commented Mar 24, 2020

@QuentinBrosse hmm it's still pretty small 🤔

Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Sh4d1 added 2 commits March 24, 2020 11:04
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Sh4d1 and others added 3 commits March 24, 2020 11:13
Co-Authored-By: Quentin Brosse <quentin.brosse@icloud.com>
Signed-off-by: Patrik Cyvoct <pcyvoct@scaleway.com>
Copy link
Contributor

@kindermoumoute kindermoumoute left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@QuentinBrosse QuentinBrosse left a comment

Choose a reason for hiding this comment

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

LGTM

@kindermoumoute kindermoumoute dismissed jerome-quere’s stale review March 25, 2020 09:06

2 reviews is enough

@kindermoumoute kindermoumoute merged commit a7fb946 into scaleway:v2 Mar 25, 2020
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.

6 participants