public icon indicating copy to clipboard operation
public copied to clipboard

Add atomic annotations to containers in openconfig-aft.

Open drautio opened this issue 3 years ago • 4 comments

On branch aft-atomic-508 Changes to be committed: modified: openconfig-aft-common.yang modified: openconfig-aft-ethernet.yang modified: openconfig-aft-ipv4.yang modified: openconfig-aft-ipv6.yang modified: openconfig-aft-mpls.yang modified: openconfig-aft-pf.yang

Add the atomic attribute to the above aft yang containers.

drautio avatar Oct 29 '21 16:10 drautio

Compatibility Report for commit f4cb642ffbcdd69bdb04bb223d2619561271cbc4: ⛔ yanglint@SO 1.10.17

OpenConfigBot avatar Oct 29 '21 16:10 OpenConfigBot

goyang/ygot's failure is due to an outdated CI image. When you retrigger the CI now it should bypass the current failure.

wenovus avatar Nov 09 '21 22:11 wenovus

Checking in with you @drautio. Would you like to move forward to address the comments?

dplore avatar Dec 14 '21 18:12 dplore

I made the requested changes. Thanks for the review!

  • Dan

drautio avatar Jan 26 '22 18:01 drautio

I'll update this PR if I can get that to work, or otherwise make a new PR.

We'll take care of this, Dan. :)

michaelore avatar Feb 15 '23 18:02 michaelore

Closed in favor of #860

dplore avatar Apr 19 '23 15:04 dplore