Skip to content

[BUG] Datacatalog enable and catalog-cache setting mismatch cause DNS #7155

@dinhxuanvu

Description

@dinhxuanvu

Flyte & Flytekit version

flyte-core v1.16.4

Describe the bug

By default, datacatalog is enabled (here) which works with the default settings for catalog-cache (here).

However, this two settings are miles apart especially the catalog-cache setting is deeply embedded inside configmaps. So if datacatalog is disabled which is not uncommon to reduce operational complexity, the default settings on catalog-cache will cause DNS failure due to datacatalog is not there. This is a gotcha moment because users may not know there is a default setting on configmaps that explicitly sets the cache entrypoint to use datacatalog.

Expected behavior

These two related configs should be linked together. If datacatalog is disabled, flyte should automatically disable the catalog-cache instead of using datacatalog service which doesn't exist.

Additional context to reproduce

Disable datacatalog without changing catalog-cache settings in configmaps setting.

Screenshots

Workflow[xxxx] failed.
RuntimeExecutionError: max number of system retry attempts [31/30] exhausted.
Last known status message: Failed to check Catalog for previous results:
DataCatalog failed to get dataset for ID
resource_type:TASK
project:"xxxx"
domain:"xxxx"
name:"xxxx"
version:"xxxx":
rpc error: code = Unavailable
desc = connection error: desc = "transport: Error while dialing: dial tcp: lookup datacatalog on x.x.x.x:x no such host"

Are you sure this issue hasn't been raised already?

  • Yes

Have you read the Code of Conduct?

  • Yes

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingflyteuntriagedThis issues has not yet been looked at by the Maintainers

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions