kne icon indicating copy to clipboard operation
kne copied to clipboard

Upgrade srl controller

Open alexmasi opened this issue 2 years ago • 4 comments

alexmasi avatar Oct 13 '22 22:10 alexmasi

/gcbrun

alexmasi avatar Oct 13 '22 22:10 alexmasi

Pull Request Test Coverage Report for Build 3246065794

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 57.964%

Totals Coverage Status
Change from base Build 3237711793: 0.0%
Covered Lines: 2329
Relevant Lines: 4018

💛 - Coveralls

coveralls avatar Oct 13 '22 22:10 coveralls

@hellt There is a regression with this update, the topo creation now hangs if the tag of the image is not of a certain form

alexmasi avatar Oct 13 '22 23:10 alexmasi

@hellt There is a regression with this update, the topo creation now hangs if the tag of the image is not of a certain form

What image tag are you referring to in particular @alexmasi ?

ADD1: got it, it is the version parsing function that detects a version out of a tag. I will log this as an issue at srl-controller repo. For the moment, the workaround is to use X.Y.Z-ddddd pattern as it is tagged originally

hellt avatar Oct 14 '22 06:10 hellt

was this superseded by another PR?

On Thu, Nov 17, 2022 at 5:33 AM Alex Masi @.***> wrote:

Closed #242 https://github.com/openconfig/kne/pull/242.

— Reply to this email directly, view it on GitHub https://github.com/openconfig/kne/pull/242#event-7828123740, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABLKV5MPFKCG7L34MDVM5HDWIVHI7ANCNFSM6AAAAAAREWSNUI . You are receiving this because you were mentioned.Message ID: @.***>

hellt avatar Nov 17 '22 17:11 hellt

yes, however I'm hitting the control license parsing from tag issues again

will send the PR for review after I get it working

alexmasi avatar Nov 17 '22 18:11 alexmasi