openvcloud / 0-templates

Apache License 2.0
1 stars 5 forks source link

data.yaml is updated with wrong data #26

Closed john-kheir closed 6 years ago

john-kheir commented 6 years ago

1- Run a blueprint to create acctest1 and vdc1 2- Run another blueprint to create acctest2 and vdc1(vdci3) (using acc2)

Actual Result

From OVC side: acctest1 is created and vdc1 is under it, then acctest2 is created without any vdc1 under it (which is okay for now ) From zrobot side: The problem is that the vdc1 data is pointing to acctest2 which shouldn't happen erer1

muhamadazmy commented 6 years ago

Can u please post the bp u used, with exact steps

john-kheir commented 6 years ago
services:
    - github.com/openvcloud/0-templates/sshkey/0.0.1__key2:
        path: '/root/.ssh/id_rsa'
    - github.com/openvcloud/0-templates/openvcloud/0.0.1__be-g8-4:
        address: 'be-g8-4.demo.greenitglobe.com'
        login: 'kheirj@itsyouonline'
        token: '****'
    - github.com/openvcloud/0-templates/vdcuser/0.0.1__kheirj:
        openvcloud: be-g8-4
        provider: itsyouonline
        email: kheirj@greenitglobe.com
    - github.com/openvcloud/0-templates/vdcuser/0.0.1__bisho:
        openvcloud: be-g8-4
        provider: itsyouonline
        email: abdelmab@greenitglobe.com
    - github.com/openvcloud/0-templates/account/0.0.1__acctest1:
        openvcloud: be-g8-4
        users:
             - accesstype: CXDRAU
               name: kheirj

    - github.com/openvcloud/0-templates/vdc/0.0.1__vdci3:
        account: acctest1
        location: be-g8-4
        users:
             - name: bisho
               accesstype: CXDRAU

actions:
    - template: github.com/openvcloud/0-templates/account/0.0.1
      actions: ['install']

    - template: github.com/openvcloud/0-templates/vdcuser/0.0.1
      actions: ['install']

    - template: github.com/openvcloud/0-templates/vdc/0.0.1
      actions: ['install']

Then i ran it again with acctest2

chrisvdg commented 6 years ago

Default behavior of update_data has changed in 0-robot that should prevent this from happening.

katia-e commented 6 years ago

Closing issue since update_data on the service is disabled. Now can update data only with update action