choodur

  • Joined on
@choodur
@choodur
  • @dtroyer 9cf0ed0
    Support multiple Cinder backend types
@choodur
  • 3d3cfad
    Add pre-install phase in stack.sh
choodur deleted branch stable/icehouse-ceph at choodur/devstack
@choodur
@choodur
choodur commented on pull request musashi-dev/aviator#54
@choodur

:+1: looks good to me. Not sure if cleaning is required after creation though (if name should be unique in OpenStack).

choodur commented on pull request musashi-dev/aviator#54
@choodur

@stefanymarie, I think you can use base_url directly instead of still parsing it.

@choodur
  • @choodur b890106
    Merge pull request #55 from lvillasica/feature/AVIATORAPI-90
  • a504aef
    Add precondition to add and assign flavor for remove flavor from proj…
  • 3 more commits »
choodur merged pull request musashi-dev/aviator#55
@choodur
AVIATORAPI-90 Delete access from private flavor
4 commits with 780 additions and 0 deletions
choodur commented on pull request musashi-dev/aviator#55
@choodur

+1.

@choodur
choodur merged pull request musashi-dev/aviator#53
@choodur
Delete Flavor API
3 commits with 810 additions and 0 deletions
choodur commented on pull request musashi-dev/aviator#53
@choodur

+1, this is okay now.

@choodur
choodur merged pull request musashi-dev/aviator#57
@choodur
AVIATORAPI-93 List tenants with access to private flavor
2 commits with 746 additions and 0 deletions
choodur commented on pull request musashi-dev/aviator#57
@choodur

Looks good to me.

@choodur
choodur merged pull request musashi-dev/aviator#59
@choodur
AVIATORAPI-89 Fix
1 commit with 27 additions and 37 deletions
choodur commented on pull request musashi-dev/aviator#59
@choodur

Good catch.

choodur commented on pull request musashi-dev/aviator#57
@choodur

Maybe having this is as a single string is better and makes it more readable, what do you think?

@choodur
choodur merged pull request musashi-dev/aviator#56
@choodur
AVIATORAPI-92 Update list flavor aviator api to show private flavors
2 commits with 467 additions and 209 deletions
choodur commented on pull request musashi-dev/aviator#56
@choodur

Looks good to me as well.

choodur commented on pull request musashi-dev/aviator#55
@choodur

@lvillasica yes, #52 is now merged.

choodur commented on pull request musashi-dev/aviator#53
@choodur

+1. this would make the test predictable and consistent.