This repository has been archived by the owner on Nov 8, 2019. It is now read-only.

Feature/90 Bootstrap command to drive chef-zero-based configuration #91

Merged

sjpatino merged 20 commits into develop from feature/90-bootstrap-command

Mar 22, 2019

Contributor

pcantera commented Mar 1, 2019 •

edited

Loading

(issue #90)

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate)

Types of changes

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to change)

Checklist

My code follows the code style of this project.
My change requires a change to the documentation.
I have updated the documentation accordingly.
I have added tests to cover my changes.
All new and existing tests passed.

pcantera added the feature label

pcantera self-assigned this

pcantera commented

View reviewed changes

utils/utils.go Outdated Show resolved Hide resolved

pcantera commented

View reviewed changes

utils/webservice.go Outdated Show resolved Hide resolved

pcantera commented

View reviewed changes

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

pcantera commented

View reviewed changes

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

pcantera commented

View reviewed changes

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

pcantera changed the title ~~Bootstrap command to drive chef-zero-based configuration (issue #90)~~ feature/Bootstrap command to drive chef-zero-based configuration (issue #90)

pcantera changed the title ~~feature/Bootstrap command to drive chef-zero-based configuration (issue #90)~~ Feature/90 Bootstrap command to drive chef-zero-based configuration

pcantera commented

View reviewed changes

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

pbanos reviewed

View reviewed changes

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

pbanos suggested changes

View reviewed changes

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

utils/utils.go Outdated Show resolved Hide resolved

utils/utils.go Show resolved Hide resolved

utils/utils.go Show resolved Hide resolved

utils/webservice.go Outdated Show resolved Hide resolved

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

pcantera commented

View reviewed changes

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

pcantera force-pushed the feature/90-bootstrap-command branch from 56ea94c to 53b68d6 Compare

March 5, 2019 15:16

pbanos reviewed

View reviewed changes

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

pcantera force-pushed the feature/90-bootstrap-command branch 3 times, most recently from 3a6a049 to 96dfacd Compare

March 5, 2019 20:04

pbanos force-pushed the feature/90-bootstrap-command branch 4 times, most recently from 50551b3 to 0762b20 Compare

March 6, 2019 17:35

pbanos approved these changes

View reviewed changes

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

bootstrapping/bootstrapping.go Outdated Show resolved Hide resolved

Pablo Cantera and others added 7 commits

March 20, 2019 17:15


          Initial commit (issue #90)

715e84c


          Renamed structs (issue #90)

be2f4f2


          Refactorized mechanism to download file (issue #90)

d87d56e


          Changed strategy for cleaning (issue #90)

87af58e


          General review and refactoring task (issue #90)

d25d8e6

- Refactored
- Implemented lines threshold for reporting while running continuos command
- Most errors in looping are only traced in order to avoid break the task
- Added chef commands
- Implemented concurrency control


          Updated routine approach (issue #90)

In addition:
- Implemented success case management for booting
- Redefined interval time parameters


          Some refactor of agent bootstrapping command including error processi…

54a1d11

…ng (issue #90)

pbanos and others added 13 commits

March 20, 2019 17:16


          Ensure a single instance of the bootstrapping command works at a time…

1bf8859

… (issue #90)


          Fix bootstrapping issues (issue #90)

5365e95


          Make bootstrapping report applied policyfiles when some fail (issue #90)

b5ced19


          Refactored single instance management (issue #90)

b8f8aaa


          Make more readable the processing error messages (issue #90)

b51bed7


          Added context management (issue #90)

8a22cbe

Used with CommandContext at uncompressing "tgz" time


          Refactor bootstrapping workspace dir management (issue #90)

5607bf0


          Added test cases (issue #90)

4556ce1


          Remove scripts boot execution in bootstrapping and move config files …

…to cio directory


          Completed test cases (issue #90)

0ec978e

- Downloading file


          Updated bootstrapping process to works in windows platform (Issue #90)

4be9ada


          Removed unused variable (issue #90)

041df51


          Have bootstrapping server command rename policyfile dir for chef runs…

2ad5989

… on windows (issue #90)

pbanos force-pushed the feature/90-bootstrap-command branch from c426535 to 2ad5989 Compare

March 20, 2019 16:21

pbanos requested review from lttito and sjpatino

March 20, 2019 16:24

lttito approved these changes

View reviewed changes

lttito left a comment

👍

sjpatino approved these changes

View reviewed changes

sjpatino left a comment

It seems right ;)

sjpatino merged commit 8fcd96f into develop

pcantera mentioned this pull request

Script execution must use the new index route and then download URLs for each attachment to obtain them #87

Closed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels