Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-25369

DNS multicast error messages

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved (View Workflow)
    • Priority: Minor
    • Resolution: Fixed
    • Component/s: core
    • Environment:
    • Similar Issues:
    • Released As:
      Jenkins 2.178

      Description

      I've seen these error messages in my console output, which comes from jmdns.

      The level of the logging messages should be toned down significantly given that these are from a relatively minor feature in Jenkins.

        Attachments

          Issue Links

            Activity

            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            LTS backporting comment: let's do more testing before considering backporting

            Show
            oleg_nenashev Oleg Nenashev added a comment - LTS backporting comment: let's do more testing before considering backporting
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            It was released in Jenkins 2.178, but we do not think backporting is justified for now

            Show
            oleg_nenashev Oleg Nenashev added a comment - It was released in Jenkins 2.178, but we do not think backporting is justified for now
            Hide
            fcojfernandez Francisco Fernández added a comment -

            I've been doing some research here and updating the library seems to fix the problem. At least is a starting point. 

            As I can see, the library was forked just because we want to use an abort method instead of the close method provided by jmdns. Moving the logic of this abort method to the core, probably we can put aside the fork and use the library instead, so any update would be easier. Opening a PR soon.

            Show
            fcojfernandez Francisco Fernández added a comment - I've been doing some research here and updating the library seems to fix the problem. At least is a starting point.  As I can see, the library was forked just because we want to use an abort  method instead of the close method provided by jmdns. Moving the logic of this abort method to the core, probably we can put aside the fork and use the library instead, so any update would be easier. Opening a PR soon.
            Hide
            oleg_nenashev Oleg Nenashev added a comment -

            It may happen even without network issues, e.g. when there is an unsupported DNS Message type coming from DNS. We use an old library version, and not all DNS Message types are supported (discovered in JENKINS-50812)

            Record type enumeration in the lib: https://github.com/jenkinsci/jmdns/blob/64f48ab717e0c8022f60d7dcb7f148f47c7f1227/src/main/java/javax/jmdns/impl/constants/DNSRecordType.java

            List of all DNS record types: https://en.wikipedia.org/wiki/List_of_DNS_record_types . Likely some types have been added in new DNS versions via RFCs. E.g. URI identifier has been added in 2015 by https://tools.ietf.org/html/rfc7553 , so no wonder it's not supported by the lib.

            There is https://github.com/jenkinsci/jmdns/pull/2 , which may resolve the issue by bumping to the new version

            Show
            oleg_nenashev Oleg Nenashev added a comment - It may happen even without network issues, e.g. when there is an unsupported DNS Message type coming from DNS. We use an old library version, and not all DNS Message types are supported (discovered in JENKINS-50812 ) Record type enumeration in the lib: https://github.com/jenkinsci/jmdns/blob/64f48ab717e0c8022f60d7dcb7f148f47c7f1227/src/main/java/javax/jmdns/impl/constants/DNSRecordType.java List of all DNS record types: https://en.wikipedia.org/wiki/List_of_DNS_record_types . Likely some types have been added in new DNS versions via RFCs. E.g. URI identifier has been added in 2015 by https://tools.ietf.org/html/rfc7553 , so no wonder it's not supported by the lib. There is https://github.com/jenkinsci/jmdns/pull/2 , which may resolve the issue by bumping to the new version
            Hide
            danielbeck Daniel Beck added a comment -

            Related to JENKINS-10160, possibly a duplicate.

            Show
            danielbeck Daniel Beck added a comment - Related to JENKINS-10160 , possibly a duplicate.

              People

              • Assignee:
                fcojfernandez Francisco Fernández
                Reporter:
                uday Udaypal Aarkoti
              • Votes:
                12 Vote for this issue
                Watchers:
                25 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: