search

project-arlo / sonic-mgmt-framework

SONiC Management Framework
Other
5 stars 15 forks source link

Deletion of leaf-list with instance doesn't have ygot strcuture populated with value #127

Closed justinejose91 closed 5 years ago

justinejose91 commented 5 years ago

When you delete a leaf-list with instance, ygot structure for APP-module doesn't contain leaf-list data populated.

justinejose91 commented 5 years ago

@mbalachandar Did you get a chance to look into this issue?

jeff-yin commented 5 years ago

@mbalachandar -- any update? Can't complete the VLAN feature without this.

mbalachandar commented 5 years ago

Hi Jeff

The ETA for this support is on coming Tuesday Oct'1 2019.

Thanks Balachandar

mbalachandar commented 5 years ago

This issue is fixed and checked in. You can pull the changes

justinejose91 commented 5 years ago

Do we have changes in master? I tested it out now, and it still doesn't have ygot structure populated with trunk Vlan value.

mbalachandar commented 5 years ago

Hi Justin

Its not in master, please check with Prabhu to pull the changes from broadcom branch.

Thanks Balachandar

justinejose91 commented 5 years ago

@PrabhuSreenivasan Could you tell me when these changes would be available in master?

bhavini-gada commented 5 years ago

@PrabhuSreenivasan - Where are you merging this change ?

jeff-yin commented 5 years ago

@justinejose91 -- did the merge get done? Can we close this issue?

sachin-brm commented 5 years ago

@Justine.Jose@dell.com Justine.Jose@dell.com Arun synced the code early this week. I think Bala's fix would have got merged already... copying him to check and confirm though...

Regards,

Sachin.

On Fri, Oct 18, 2019 at 11:36 AM Jeff Yin notifications@github.com wrote:

@justinejose91 https://github.com/justinejose91 -- did the merge get done? Can we close this issue?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/project-arlo/sonic-mgmt-framework/issues/127?email_source=notifications&email_token=AMNPE22NSFOEUJZ2QRI3VOLQPFG6DA5CNFSM4IYEIDWKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBSZYPA#issuecomment-543530044, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMNPE25OOKKJC7A3L3T2HALQPFG6DANCNFSM4IYEIDWA .

justinejose91 commented 5 years ago

This is merged. @sachin-brm @jeff-yin I verified it. Closing the issue.