You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficcontrol.apache.org by GitBox <gi...@apache.org> on 2021/04/09 22:23:34 UTC

[GitHub] [trafficcontrol] zrhoffman opened a new pull request #5731: Set server XMPPID to hostname if empty

zrhoffman opened a new pull request #5731:
URL: https://github.com/apache/trafficcontrol/pull/5731


   This PR backports #5725 to 5.1.x
   
   * Generate XMPPID on server update if it is empty
   
   * Hyphenate traffic-ops.yml
   
   * Simplify condition
   
   * Migration sets XMPPID to hostname if XMPPID is null
   
   * Do not persistently set XMPPID on PUT if it is empty
   
   * Remove xmpp_id from SQL query
   
   * Include hostname in warning
   
   * Use newer definition of XMPPID for API v1
   
   * More accurate changelog entry
   
   (cherry picked from commit f599571c9e076d1176fed1ba966d0fc43013829c)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] zrhoffman commented on pull request #5731: Set server XMPPID to hostname if empty

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on pull request #5731:
URL: https://github.com/apache/trafficcontrol/pull/5731#issuecomment-827321755


   f599571c9e is from the master branch.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] zrhoffman commented on pull request #5731: Set server XMPPID to hostname if empty

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on pull request #5731:
URL: https://github.com/apache/trafficcontrol/pull/5731#issuecomment-827296127


   Rebased to fix a merge conflict in CHANGELOG.md


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] ocket8888 merged pull request #5731: Set server XMPPID to hostname if empty

Posted by GitBox <gi...@apache.org>.
ocket8888 merged pull request #5731:
URL: https://github.com/apache/trafficcontrol/pull/5731


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] zrhoffman commented on pull request #5731: Set server XMPPID to hostname if empty

Posted by GitBox <gi...@apache.org>.
zrhoffman commented on pull request #5731:
URL: https://github.com/apache/trafficcontrol/pull/5731#issuecomment-817004683


   Removed the DB migration that was included in #5725.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [trafficcontrol] ocket8888 commented on pull request #5731: Set server XMPPID to hostname if empty

Posted by GitBox <gi...@apache.org>.
ocket8888 commented on pull request #5731:
URL: https://github.com/apache/trafficcontrol/pull/5731#issuecomment-827310725


   #5725 is already in 5.1.x as of f599571c9e076d1176fed1ba966d0fc43013829c


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org