Skip to content

Add container-canary.#6

Open
Dan Allan (danielballan) wants to merge 15 commits into
NSLS2:mainfrom
danielballan:canary
Open

Add container-canary.#6
Dan Allan (danielballan) wants to merge 15 commits into
NSLS2:mainfrom
danielballan:canary

Conversation

@danielballan

@danielballan Dan Allan (danielballan) commented Dec 18, 2023

Copy link
Copy Markdown
Collaborator

Adds conatiner-canary to validate images on PR or release, before publishing.

This is modeled on the setup used in Tiled.

Note that if / is protected by an API key, a valid API key will need to be added here. I suppose it may also need a Mongo container running....

https://github.com/stuartcampbell/nsls2api/pull/6/files#diff-0699ff2a471369440b57203f855a77ca56d48dd98bbd11997fdcd144e150bba8R12-R14

@danielballan

Copy link
Copy Markdown
Collaborator Author

The container does not build:

#13 2.932       LookupError: Error getting the version from source `vcs`: setuptools-scm was unable to detect version for /code.

@stuartcampbell

Copy link
Copy Markdown
Collaborator

The container does not build:

#13 2.932       LookupError: Error getting the version from source `vcs`: setuptools-scm was unable to detect version for /code.

I wasn't copying the .git into the container so it couldn't figure out the version - should be fixed now

@danielballan

Copy link
Copy Markdown
Collaborator Author

Now canary is running, but its checks fail. My guess is that the container has some dependencies (a database?) without which it may not start.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants