Hey Everyone! I am trying to fix some issues with ...
# singer-tap-development
c
Hey Everyone! I am trying to fix some issues with the
tap-airbyte-wrapper
repo but I am getting the following error when trying to run some tests: But the following is causing an error. It seems like the logging file for metrics isn't being found, so my tests aren't running. Anyone have any experience with this?
/site-packages/singer_sdk/metrics.py:391: in _load_yaml_logging_config
https://github.com/MeltanoLabs/tap-airbyte-wrapper/actions/runs/10150901636/job/28069210152#step:5:103
👀 1
e
I have not. Did this happen in also before you singer-sdk to 0.38.0 in that PR?
c
It is also happening on 0.36.1. It seems like the tests on this repo have been broken for quite some time.
e
Yeah, I'm unfortunately not too familiar with the repo but I'm happy to help you debug 😅
c
It is this function that is throwing the error (in the singer SDK):
Copy code
def _get_default_config() -> t.Any:  # noqa: ANN401
    """Get a logging configuration.

    Returns:
        A logging configuration.
    """
    log_config_path = get_package_files("singer_sdk").joinpath("default_logging.yml")
    return _load_yaml_logging_config(log_config_path)
e
Yeah but what's weird is that error doesn't happen with other taps
c
Not sure if it has anything to do with the fact that this tap creates additional virtual environments
e
Another thing is it only fails in Python 3.10 (and above presumably?)
The stdlib importlib.resources in 3.9 does seem to cause some problems, so the next sdk version will use the third-party backport for 3.8 and 3.9: https://github.com/meltano/sdk/blob/5eb12353f175c0b64bcb26e52862301bba6f82d6/pyproject.toml#L47
c
@Edgar Ramírez (Arch.dev) added
e
Thanks, so the 3.9 tests pass. 3.8 has a syntax error.
c
And 3.10 has the error with the missing file....interesting