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

Agones SDK with Unreal Engine Example #2241

Closed
wants to merge 15 commits into from
Closed

Agones SDK with Unreal Engine Example #2241

wants to merge 15 commits into from

Conversation

WilSimpson
Copy link
Contributor

@WilSimpson WilSimpson commented Aug 27, 2021

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:
There have been requests for a full Unreal Engine example. This example bundles a complete Unreal Engine 2.26.2 - Third Person Shooter example game. Shows how to include Agones on only server builds and not offline lobbies. Shows how to use Agones in C++ and in blueprints. Ready to build, package and deploy.

Which issue(s) this PR fixes:

Closes #2217

Special notes for your reviewer:
This is incomplete and posted for early review and further discussion. Here is what is currently done and plan to be work on:

  • Documentation
  • Implement Agones in C++
  • Implement Agones in Blueprints
  • Test for implementation bugs
  • Test for server/client bugs with packaging
  • Test if all necessary content is included
  • Test documentation for accuracy from fresh pull

Some decisions further decisions:

  • Do we include generated source code from Unreal? (~1.3 GB of space saved by not including)
  • Do we include compiled binaries? (Unknown space currently)

Any other requests for features in the example?

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: WilSimpson
To complete the pull request process, please assign dzlier-gcp after the PR has been reviewed.
You can assign the PR to them by writing /assign @dzlier-gcp in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@WilSimpson WilSimpson changed the title Dev unrealexample Agones SDK with Unreal Engine Example Aug 27, 2021
@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 023f7d2a-269d-417f-a889-bd4b208e271c

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

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/2241/head:pr_2241 && git checkout pr_2241
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.17.0-f4cc4d0

@Ludea
Copy link
Contributor

Ludea commented Sep 1, 2021

FirstPersonExampleMap is missing

@WilSimpson
Copy link
Contributor Author

WilSimpson commented Sep 1, 2021

I renamed them to EntryLevel and ServerLevel to hopefully improve clarity of what each level is used for. I noticed I did not set this in the settings, so I have done so now. Thanks!

@agones-bot
Copy link
Collaborator

Build Failed 😱

Build Id: 3166a29c-e17e-4e39-8ba8-72f11de2eeaf

To get permission to view the Cloud Build view, join the agones-discuss Google Group.

@google-cla
Copy link

google-cla bot commented Sep 2, 2021

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for all the commit author(s) or Co-authors. If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.
In order to pass this check, please resolve this problem and then comment @googlebot I fixed it.. If the bot doesn't comment, it means it doesn't think anything has changed.

ℹ️ Googlers: Go here for more info.

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: d63dcd48-6fec-4a27-8bee-be82a165eae8

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

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/2241/head:pr_2241 && git checkout pr_2241
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.17.0-0f393ac

@markmandel
Copy link
Collaborator

Just came to say that I applaud this effort, even if I have no idea what is going on 😄

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 663ed227-dae1-432d-94e9-16db86a64d00

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

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/2241/head:pr_2241 && git checkout pr_2241
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.17.0-87c0f7b

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: c50b8858-cef1-4e0c-a911-43277d0982e4

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

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/2241/head:pr_2241 && git checkout pr_2241
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.17.0-f4cc4d0

@agones-bot
Copy link
Collaborator

Build Succeeded 👏

Build Id: 6fdaeb99-1ac8-4427-a9e1-e3500b4476f0

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

A preview of the website (the last 30 builds are retained):

To install this version:

  • git fetch https://github.com/googleforgames/agones.git pull/2241/head:pr_2241 && git checkout pr_2241
  • helm install ./install/helm/agones --namespace agones-system --name agones --set agones.image.tag=1.18.0-917eca3

RUN apt-get update \
&& apt-get upgrade -y

RUN adduser server
Copy link
Contributor

Choose a reason for hiding this comment

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

Use useradd instead of adduser, it avoid some interactive message

kind: GameServer
metadata:
generateName: "unrealexample-"
spec:
Copy link
Contributor

Choose a reason for hiding this comment

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

template key is missing

@roberthbailey
Copy link
Member

Based on "This is incomplete and posted for early review and further discussion" from the initial comment I've changed this to a draft pull request. Please move it back to active when it is complete. Thanks!


RUN apt-get update \
&& apt-get upgrade -y

Copy link
Contributor

Choose a reason for hiding this comment

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

You have to install xdg-user-dirs


EXPOSE 7777/udp

ENTRYPOINT ["/home/server/server/AgonesExampleServer.sh"]
Copy link
Contributor

Choose a reason for hiding this comment

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

Use CMD instead

@roberthbailey roberthbailey added the feature-freeze-do-not-merge Only eligible to be merged once we are out of feature freeze (next full release) label Oct 5, 2021
@roberthbailey roberthbailey removed the feature-freeze-do-not-merge Only eligible to be merged once we are out of feature freeze (next full release) label Oct 12, 2021
@markmandel
Copy link
Collaborator

It's been a long time since this was worked on - should we close it for now and revisit at a later date?

@markmandel markmandel added the stale Pending closure unless there is a strong objection. label Nov 8, 2021
@markmandel
Copy link
Collaborator

So it's been 7 days since I suggested we close this, and it doesn't seem like we've had any objections.

So I'm going to close this. Please open a new PR if you want to revisit 😄 we would love to have this example.

@markmandel markmandel closed this Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes do-not-merge/work-in-progress size/XXL stale Pending closure unless there is a strong objection.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unreal Example
6 participants