[ 
https://issues.apache.org/jira/browse/HDDS-7817?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17703824#comment-17703824
 ] 

István Fajth commented on HDDS-7817:
------------------------------------

As discussed in the PR, [~erose] will add a solution to HDDS-6633 in the coming 
days, that effort is blocking this one, as the current tests are affected by 
this change, and it should just go to the 1.4.0 upgrade tests.

Also we agreed offline, that this patch should be split to two, as 
RATIS_DATASTREAM port has the same problem, but it is not handled in this one 
and as it was added earlier, this one will require a newer metadata layout 
version than the one that fixes the RATIS_DATASTREAM compatibility issue.

So I have created HDDS-8239 and set up JIRA relations to properly track the 
effort further on.

> Add HTTP and HTTPS ports to DatanodeDetails
> -------------------------------------------
>
>                 Key: HDDS-7817
>                 URL: https://issues.apache.org/jira/browse/HDDS-7817
>             Project: Apache Ozone
>          Issue Type: Improvement
>          Components: Ozone Datanode, SCM
>            Reporter: István Fajth
>            Assignee: Zita Dombi
>            Priority: Major
>              Labels: knox, pull-request-available
>
> Currently we do not have the HTTP and HTTPS ports in the DatanodeDetails 
> object, in order to implement HDDS-7816 we need to have the port information 
> in the DatanodeDetails, so that SCM will know where to direct the links 
> within its new UI element.
> For adding HTTP and HTTPS ports:
> 1. we need to modify the enumeration and add these two
> 2. DataNode initialization should set the port on the DatanodeDetails object 
> used by DN
> Hint: the DatanodeDetails is created in the start() method of 
> HDDSDatanodeService, while the ports can be fetched from the configuration 
> similarly as BaseHttpServer fetches the information.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to