No registry lookup in cli

[butonic]

I changed the index command to use a go grpc client instead of a go micro client. It now has a grpc-addr that defaults to 127.0.0.1:9220, the search service grpc port.

It also allows using the index command when the micro service registry is set to memory.

I added the grpc client stubs to the protobuf generation in a separate commit. The change itself is small.

The only question I have is: what is a better name for the cli grpc-addr flag. maybe endpoint? we might be using unix:/path/to/socket at some point.

Related: #2715.

[codacy-production]

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 264 complexity · 3 duplication

Metric	Results
Complexity	264
Duplication	3

View in Codacy

_{NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.}

[aduffeck]

+1 for "endpoint".

Tests are still failing, but the change itself looks good to me.

[butonic]

meh, the tests actually execute the command ...

  Scenario: reindex all spaces                                               # /woodpecker/src/github.com/opencloud-eu/opencloud/tests/acceptance/features/cliCommands/searchReIndex.feature:15
    When the administrator reindexes all spaces using the CLI                # CliContext::theAdministratorReindexesAllSpacesUsingTheCli()
    Then the command should be successful                                    # CliContext::theCommandShouldBeSuccessful()
      Failed asserting that two strings are identical.
      --- Expected
      +++ Actual
      @@ @@
      -'OK'
      +'ERROR'
    When user "Alice" searches for "textfile.txt" using the WebDAV API       # SearchContext::userSearchesUsingWebDavAPI()
    Then the HTTP status code should be "207"                                # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And the search result of user "Alice" should contain only these entries: # FeatureContext::thePropfindResultShouldContainOnlyEntries()
      | textfile.txt |

  Scenario: reindex a space                                                                           # /woodpecker/src/github.com/opencloud-eu/opencloud/tests/acceptance/features/cliCommands/searchReIndex.feature:24
    Given using spaces DAV path                                                                       # FeatureContext::usingOldOrNewOrSpacesDavPath()
    And user "Alice" has created the following tags for file "textfile.txt" of the space "new-space": # TagContext::theUserHasCreatedFollowingTags()
      | tag1 |
    And user "Alice" has removed the following tags for file "textfile.txt" of space "new-space":     # TagContext::userHAsRemovedTheFollowingTagsForFileOfSpace()
      | tag1 |
    When the administrator reindexes a space "new-space" using the CLI                                # CliContext::theAdministratorReindexesASpaceUsingTheCli()
    Then the command should be successful                                                             # CliContext::theCommandShouldBeSuccessful()
      Failed asserting that two strings are identical.
      --- Expected
      +++ Actual
      @@ @@
      -'OK'
      +'ERROR'
  Scenario: reindex all spaces                                               # /woodpecker/src/github.com/opencloud-eu/opencloud/tests/acceptance/features/cliCommands/searchReIndex.feature:15
    When the administrator reindexes all spaces using the CLI                # CliContext::theAdministratorReindexesAllSpacesUsingTheCli()
    Then the command should be successful                                    # CliContext::theCommandShouldBeSuccessful()
      Failed asserting that two strings are identical.
      --- Expected
      +++ Actual
      @@ @@
      -'OK'
      +'ERROR'
    When user "Alice" searches for "textfile.txt" using the WebDAV API       # SearchContext::userSearchesUsingWebDavAPI()
    Then the HTTP status code should be "207"                                # FeatureContext::thenTheHTTPStatusCodeShouldBe()
    And the search result of user "Alice" should contain only these entries: # FeatureContext::thePropfindResultShouldContainOnlyEntries()
      | textfile.txt |

  Scenario: reindex a space                                                                           # /woodpecker/src/github.com/opencloud-eu/opencloud/tests/acceptance/features/cliCommands/searchReIndex.feature:24
    Given using spaces DAV path                                                                       # FeatureContext::usingOldOrNewOrSpacesDavPath()
    And user "Alice" has created the following tags for file "textfile.txt" of the space "new-space": # TagContext::theUserHasCreatedFollowingTags()
      | tag1 |
    And user "Alice" has removed the following tags for file "textfile.txt" of space "new-space":     # TagContext::userHAsRemovedTheFollowingTagsForFileOfSpace()
      | tag1 |
    When the administrator reindexes a space "new-space" using the CLI                                # CliContext::theAdministratorReindexesASpaceUsingTheCli()
    Then the command should be successful                                                             # CliContext::theCommandShouldBeSuccessful()
      Failed asserting that two strings are identical.
      --- Expected
      +++ Actual
      @@ @@
      -'OK'
      +'ERROR'