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

Inconsistent state #111

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

chemistry-sourabh
Copy link
Contributor

@chemistry-sourabh chemistry-sourabh commented Apr 25, 2017

@CCI-MOC/bmi-breakers initial sneak peek into the inconsistent state changes #57 . Provision is done and also wrote some test cases, review them and suggest some more cases (Plan to write 2 more).

Deprovision in theory is done need to do it in a full proof manner and write test cases.

@chemistry-sourabh
Copy link
Contributor Author

Finished Testcases can be reviewed by @CCI-MOC/bmi-breakers. Only don't merge yet as I am facing a weird issue which is not happening always.

@chemistry-sourabh
Copy link
Contributor Author

@naved001 I rebased this. Once check if your registration error is happening.

@naved001 naved001 changed the base branch from dev to master September 8, 2017 12:04
@apoorvemohan
Copy link
Collaborator

@naved001 @chemistry-sourabh should we close this?

@naved001
Copy link
Contributor

@mihirborkar @apoorvemohan I think we should take this PR and try to get it merged since it handles a lot of failures!

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

4 participants