Skip to content
This repository was archived by the owner on Jul 17, 2024. It is now read-only.

Fix some CI tests #238

Merged
merged 10 commits into from
Jul 11, 2022
Merged

Fix some CI tests #238

merged 10 commits into from
Jul 11, 2022

Conversation

plotnick
Copy link
Contributor

@plotnick plotnick commented Jul 8, 2022

Fixes some (but not all) of #204. Requires a new docker image from Omicron#1384 to allow Nexus to even potentially run (missing dependencies), and updates the omicron.toml configuration so that Nexus can run ([deployment] changes).

@plotnick
Copy link
Contributor Author

plotnick commented Jul 8, 2022

Adding a dbg!(&err) on a failing test (test_cmd_auth) yields:

[src/cmd_auth.rs:635] &err = reqwest::Error {
    kind: Request,
    url: Url {
        scheme: "http",
        cannot_be_a_base: false,
        username: "",
        password: None,
        host: Some(
            Domain(
                "localhost",
            ),
        ),
        port: Some(
            8888,
        ),
        path: "//session/me",
        query: None,
        fragment: None,
    },
    source: hyper::Error(
        Connect,
        ConnectError(
            "tcp connect error",
            Os {
                code: 111,
                kind: ConnectionRefused,
                message: "Connection refused",
            },
        ),
    ),
}

@plotnick
Copy link
Contributor Author

plotnick commented Jul 9, 2022

docker logs nexus shows:

nexus: error while loading shared libraries: libxmlsec1-openssl.so.1: cannot open shared object file: No such file or directory

So the problem is that Nexus isn't starting because it doesn't have all of its dependencies installed. In particular, xmlsec1 is a new(ish) dependency, added with the recent SAML work.

@plotnick plotnick force-pushed the ci branch 2 times, most recently from 3be3325 to aa9159c Compare July 9, 2022 01:18
@plotnick plotnick changed the title CI experiments Fix some CI tests Jul 9, 2022
@plotnick plotnick marked this pull request as ready for review July 9, 2022 02:20
@plotnick plotnick mentioned this pull request Jul 9, 2022
2 tasks
Copy link
Collaborator

@karencfv karencfv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NICE! Thanks @plotnick for fixing the auth tests!

Should be ok now that Omicron#1384 has landed.
@plotnick plotnick merged commit 6080959 into main Jul 11, 2022
@plotnick plotnick deleted the ci branch July 11, 2022 16:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants