Skip to content
This repository has been archived by the owner on Apr 27, 2022. It is now read-only.

Remove dev install #171

Merged
merged 3 commits into from Aug 22, 2018
Merged

Remove dev install #171

merged 3 commits into from Aug 22, 2018

Conversation

naved001
Copy link
Contributor

@naved001 naved001 commented Jun 29, 2018

We have the scripts to setup ceph and HIL on a different repo, and we have the ansible playbook here to deploy bmi. All that obviates the need for the scripts to setup a whole development environment.

Fixes #159

@naved001
Copy link
Contributor Author

oh and yeah, it moves the installation stuff out of the scripts folder.

Copy link
Collaborator

@ianballou ianballou left a comment

Choose a reason for hiding this comment

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

All good, just a comment.

@@ -20,6 +20,15 @@ If you're looking to code, there are a few ways to help:
* Reviewing [Pull Requests](https://github.com/CCI-MOC/ims/pulls)
* Discussions on open github issues.


## Settin up a dev environment
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think you forgot a 'g' here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good catch, I'm talking like a G.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed.

@naved001
Copy link
Contributor Author

@apoorvemohan @mihirborkar can either of you review this? This is pretty straightforward.

Copy link
Member

@jeremyfreudberg jeremyfreudberg left a comment

Choose a reason for hiding this comment

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

looks fine

* This removes the scripts that installed ceph, hil, bmi and all related packages.
@naved001 naved001 merged commit 45f77bf into CCI-MOC:master Aug 22, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants