Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docker image entrypoint is now /quilkin #607

Merged
merged 1 commit into from
Sep 28, 2022

Conversation

markmandel
Copy link
Contributor

What type of PR is this?

Uncomment only one /kind <> line, press enter to put that in a new line, and remove leading whitespace from that line:

/kind breaking
/kind bug
/kind cleanup
/kind documentation

/kind feature

/kind hotfix

What this PR does / Why we need it:

This makes life easier for running Quilkin in different modes.

  • Changed the entrypoint to /quilkin
  • Fixed cloudbuild.yaml tests.
  • Updates to agones integration tests
  • Updated container documentation
  • Update Xonotic example, including converting the simple sidecar example to command line arguments.

Which issue(s) this PR fixes:

Closes #583

Special notes for your reviewer:

@markmandel markmandel added kind/feature New feature or request area/operations Installation, updating, metrics etc labels Sep 27, 2022
@markmandel markmandel force-pushed the image/entrypoint branch 3 times, most recently from 9229bce to cfb13a2 Compare September 28, 2022 00:39
* Changed the entrypoint to /quilkin
* Fixed cloudbuild.yaml tests.
* Updates to agones integration tests
* Updated container documentation
* Update Xonotic example, including converting the simple sidecar
  example to command line arguments.

Closes googleforgames#583
@quilkin-bot
Copy link
Collaborator

Build Succeeded 🥳

Build Id: 99c7d307-1c76-46e4-8fb9-8e30f0e4eb43

The following development images have been built, and will exist for the next 30 days:

To build this version:

git fetch [email protected]:googleforgames/quilkin.git pull/607/head:pr_607 && git checkout pr_607
cargo build

@markmandel markmandel merged commit 736e413 into googleforgames:main Sep 28, 2022
@markmandel markmandel deleted the image/entrypoint branch September 28, 2022 15:52
XAMPPRocky pushed a commit that referenced this pull request Oct 10, 2022
* Changed the entrypoint to /quilkin
* Fixed cloudbuild.yaml tests.
* Updates to agones integration tests
* Updated container documentation
* Update Xonotic example, including converting the simple sidecar
  example to command line arguments.

Closes #583
XAMPPRocky pushed a commit that referenced this pull request Oct 10, 2022
* Changed the entrypoint to /quilkin
* Fixed cloudbuild.yaml tests.
* Updates to agones integration tests
* Updated container documentation
* Update Xonotic example, including converting the simple sidecar
  example to command line arguments.

Closes #583
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/operations Installation, updating, metrics etc kind/feature New feature or request size/m
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker image entrypoint should just be "/quilkin"
3 participants