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

Use Go 1.13.1 runtime on GCP for cloud functions #17180

Merged
merged 6 commits into from
Mar 24, 2020

Conversation

kvch
Copy link
Contributor

@kvch kvch commented Mar 23, 2020

What does this PR do?

This PR changes the runtime of Google Cloud Functions provided by Functionbeat to 1.13.1. Furthermore, a test is added to Travis CI to check if the cloud functions can be built using Go 1.13.1.

One of the limitations of this PR is that no check is added to Jenkins yet.

Why is it important?

Previously, we had no check to catch if we had broken something in the GCP functions.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
    - [ ] I have made corresponding changes to the documentation
    - [ ] I have made corresponding change to the default configuration files
    - [ ] I have added tests that prove my fix is effective or that my feature works

Related issues

@kvch kvch added [zube]: In Review Team:Services (Deprecated) Label for the former Integrations-Services team labels Mar 23, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations-services (Team:Services)

@kvch kvch added the needs_backport PR is waiting to be backported to other branches. label Mar 23, 2020
x-pack/functionbeat/magefile.go Outdated Show resolved Hide resolved
x-pack/functionbeat/manager/gcp/template_builder.go Outdated Show resolved Hide resolved
Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

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

LGTM. Can you rebase on top of my recent changes to fix the one functionbeat test and re-run the Jenkinsfile pipeline to test your changes there.

@@ -21,7 +21,7 @@ import (
)

const (
runtime = "go111" // Golang 1.11
runtime = "go113" // Golang 1.11
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
runtime = "go113" // Golang 1.11
runtime = "go113" // Golang version.

@kvch kvch force-pushed the fix-functionbeat-gcp-functions branch from 74286eb to 499d22f Compare March 24, 2020 15:43
Copy link
Contributor

@mikemadden42 mikemadden42 left a comment

Choose a reason for hiding this comment

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

LGTM

kvch and others added 4 commits March 24, 2020 16:47
Co-Authored-By: Victor Martinez <victormartinezrubio@gmail.com>
Co-Authored-By: Victor Martinez <victormartinezrubio@gmail.com>
@kvch kvch force-pushed the fix-functionbeat-gcp-functions branch from 0aeec83 to d17fb64 Compare March 24, 2020 15:47
@kvch kvch merged commit b82b9db into elastic:master Mar 24, 2020
kvch added a commit to kvch/beats that referenced this pull request Mar 24, 2020
This PR changes the runtime of Google Cloud Functions provided by Functionbeat to 1.13.1. Furthermore, a test is added to the CIs to check if the cloud functions can be built using Go 1.13.1.

Previously, we had no check to catch if we had broken something in the GCP functions.

- Closes elastic#16827

(cherry picked from commit b82b9db)
@kvch kvch added v7.7.0 and removed needs_backport PR is waiting to be backported to other branches. labels Mar 24, 2020
kvch added a commit that referenced this pull request Mar 25, 2020
This PR changes the runtime of Google Cloud Functions provided by Functionbeat to 1.13.1. Furthermore, a test is added to the CIs to check if the cloud functions can be built using Go 1.13.1.

Previously, we had no check to catch if we had broken something in the GCP functions.

- Closes #16827

(cherry picked from commit b82b9db)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Services (Deprecated) Label for the former Integrations-Services team v7.7.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Functionbeat] Cannot deploy pubsub function on GCP
6 participants