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

Revert /system/archetype/os changes #181

Merged
merged 3 commits into from
Jun 5, 2018
Merged

Conversation

jrha
Copy link
Member

@jrha jrha commented Jun 5, 2018

These changes were included as part of #142, but were not necessary for that use-case. The impact of these changes was underestimated and as the new types appear to be unused in the community the change should be reverted ASAP.

Fixes #179.
Fixes quattor/aquilon#93.

@jrha jrha added this to the 18.6 milestone Jun 5, 2018
@jrha jrha changed the title Revert /system/archetype/os from dict to string Revert /system/archetype/os changes Jun 5, 2018
@jrha jrha requested review from ned21 and jouvin June 5, 2018 09:31
Copy link
Contributor

@jouvin jouvin left a comment

Choose a reason for hiding this comment

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

No problem for me but should be merged before #177 probably or the os and model should be made mandatory

@jrha
Copy link
Member Author

jrha commented Jun 5, 2018

I'd like to just keep this PR as a reversion, combining changes is what got us into this mess.

@jouvin
Copy link
Contributor

jouvin commented Jun 5, 2018

I agree. It was my point. We should reverse the "mess" first and rebase #177 on it rather than the opposite...

@ned21
Copy link
Contributor

ned21 commented Jun 5, 2018

Shall I merge?

@jouvin jouvin merged commit 1ed9153 into quattor:master Jun 5, 2018
@jrha jrha deleted the archetype_os branch June 5, 2018 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants