[TRNSPRTPCE-701] karaf client fails in karaf-4.3.8 and 4.4.2 Created: 09/Nov/22 Updated: 20/Sep/23 Resolved: 20/Sep/23 |
|
| Status: | Verified |
| Project: | transportpce |
| Component/s: | tapi |
| Affects Version/s: | Sulfur, Chlorine, Argon |
| Fix Version/s: | Potassium |
| Type: | Bug | Priority: | Highest |
| Reporter: | Gilles Thouenon | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| External issue ID: | https://issues.apache.org/jira/browse/KARAF-7607 |
| Description |
|
Sulfur-SR3 and Clorine-SR1 are respectively using karaf-4.3.8 and karaf-4.4.2 In these versions of karaf, when we try to launch the client (./karaf/target/assembly/bin/client), it fails with the following message:
This feature is used in transportpce tapi functional tests to install the odl-transportpce-tapi optional feature. Since the tapi feature can no longer be installed, all tapi functional tests are failing on the gate (https://jenkins.opendaylight.org/releng/job/transportpce-tox-verify-sulfur/1372/consoleFull)
Karaf client is still failing with karaf-4.4.2, even if the exception is better managed:
|
| Comments |
| Comment by Gilles Thouenon [ 09/Nov/22 ] |
|
1375 [main] DEBUG org.apache.sshd.client.SshClient - close(SshClient[d6da883])[Immediately] closed |
| Comment by Robert Varga [ 09/Nov/22 ] |
|
The failing check was introduced in https://issues.apache.org/jira/browse/SSHD-1244 . From quick glance I have not found anything fishy in SSHD itself, but Karaf's client is initializing the input in client/src/main/java/org/apache/karaf/client/Main.java:267: channel.setPtyModes(modes); can you look at modes here with a debugger? |
| Comment by Robert Varga [ 09/Nov/22 ] |
|
Alternatively, we really need to know the PtyMode key which has that -1 associated and then track down where it is set. |
| Comment by Robert Varga [ 09/Nov/22 ] |
|
And we'll need a Karaf bug, I think because JLine reports -1 for characters it does not find: https://github.com/jline/jline3/blob/92a63e1494d7e24a9f36cf639257b181080a535f/terminal/src/main/java/org/jline/terminal/Attributes.java#L292 and karaf is not filtering those. Now we need to figure out which attributes are missing and why – because perhaps this can be solved with setting TERM via https://github.com/jline/jline3/wiki/Terminals – I don't really know, though. |
| Comment by Robert Varga [ 10/Nov/22 ] |
|
Upstream issue: https://issues.apache.org/jira/browse/KARAF-7607 |
| Comment by Robert Varga [ 10/Nov/22 ] |
|
Upstream (partial?) fix: https://github.com/apache/karaf/pull/1680 We still need to try to work this around with TERM. |
| Comment by Robert Varga [ 10/Nov/22 ] |
|
Hopefully the unavailable control char is not one of the three mentioned here: https://github.com/apache/karaf/pull/1680/commits/96128a63c7c71f613b88ca05017c7e7e75b86d85 |
| Comment by Gilles Thouenon [ 10/Nov/22 ] |
|
Thank you Robert to take care of this issue. I've just tested your karaf PR and obtained the following messge now:
|
| Comment by Robert Varga [ 10/Nov/22 ] |
|
Ah, so it is |
| Comment by Gilles Thouenon [ 10/Nov/22 ] |
|
Good news! |
| Comment by Robert Varga [ 21/Nov/22 ] |
|
Thanks for testing Unfortunately it seems we are not getting a new Karaf for a couple of weeks. I am loathe to downgrade, but it seems the best short-term option to get this unblocked. Q: which release train is the most pressing? Cl SR1? Su SR2? |
| Comment by Gilles Thouenon [ 22/Nov/22 ] |
|
From TPCE side, no pressing at all for any of these releases. From ODL release schedule, it seems Su SR2 is rather urgent if we don't want make the delay worse... |
| Comment by Daniel De La Rosa [ 23/Nov/22 ] |
|
you meant Su SR3 or I'm missing something ? |
| Comment by Gilles Thouenon [ 25/Nov/22 ] |
|
Yes of course, this is Su SR3. In TransportPCE, we have overcomed this issue by switching on lighty build as the runtime for all TransportPCE functional tests on the gate instead of karaf. By this way, this issue is no longer blocking for Su-SR3 nor Cl-SR1. |