Add support for inclusive naming #156
Conversation
Signed-off-by: Thomas Farr <tsfarr@amazon.com>
Signed-off-by: Thomas Farr <tsfarr@amazon.com>
Signed-off-by: Thomas Farr <tsfarr@amazon.com>
Signed-off-by: Thomas Farr <tsfarr@amazon.com>
Xtansia
force-pushed
the
feat/inclusive-naming
branch
from
ee0a2d2 to
35524fc
Compare
| @@ -46,7 +46,7 @@ func TestBuild_zipfileUrl(t *testing.T) { | |||
| "build_id": "1.0.0-ab7cd914", | |||
| "projects": { | |||
| "opensearch": { | |||
| "branch": "master", | |||
| "branch": "main", | |||
There was a problem hiding this comment.
@Xtansia Do we know if the branch change has happened? I was unable to find the artifact listed here.
There was a problem hiding this comment.
@harshavamsi This is just test data, not actually used other than to validate some logic that pulls out the archive url.
|
@dblock @VachaShah @wbeckler This looks good to me, bar the one flaky integration test and whitesource CVE that I'm not sure are relevant here. |
|
Please open an issue for the flaky test failure in https://github.com/opensearch-project/opensearch-go/actions/runs/3012819413/jobs/4841298373? |
@dblock Looks like Vacha already encountered and created an issue for it: #163 |
Description
cluster_manager_timeoutparameter, deprecatingmaster_timeout/_cat/cluster_managerendpoint, deprecating/_cat/mastercluster_managernode role in the discovery logicWithLocalmethods to use cluster-manager terminology.Issues Resolved
Resolves #121
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.