Skip to content

Add container-canary.#6

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

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

Conversation

@danielballan
Copy link
Copy Markdown
Collaborator

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

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