ci: run test on Windows #15570

kuisathaverat · 2020-01-15T11:00:57Z

test purposes, not ready for review yet

related to elastic/observability-dev#505

job: https://beats-ci.elastic.co/job/Beats/job/beats-beats-mbp/view/change-requests/job/PR-15570/

kuisathaverat · 2020-01-15T15:29:33Z

jenkins run the tests

kuisathaverat · 2020-01-16T11:51:05Z

@andrewkroh the pipeline part is done, now we have to review some test that fails on windows and see if we want to run then on windows or not, How we manage it?

cc @elastic/observablt-robots

kuisathaverat · 2020-01-22T14:08:06Z

This job has some tests failing https://beats-ci.elastic.co/blue/organizations/jenkins/Beats%2Fbeats-beats-mbp/detail/PR-15570/14/tests

andrewkroh

Everytime I read these pipelines I learning a bit more 😄. For the windows targets I'd reduce down to just to go unit tests to keep it simple. After we get that working then we can work on introducing in the system tests in a separate PR.

.ci/scripts/install-tools.bat

Jenkinsfile

andrewkroh

The new conditions look great.

We might not have an x-pack/winlogbeat build on Travis, but we do want a Windows build of this on the Jenkins pipeline (and already have a build of this on Jenkins). It probably doesn't exist on Travis since we don't use any Windows workers on Travis.

The other three Beats you mention might not be setup yet with an independent Makefile/magefile in their x-pack directory so don't worry about them for now.

Jenkinsfile

…t is needed

kuisathaverat · 2020-02-05T11:23:39Z

We might not have an x-pack/winlogbeat build on Travis, but we do want a Windows build of this on the Jenkins pipeline (and already have a build of this on Jenkins). It probably doesn't exist on Travis since we don't use any Windows workers on Travis.

I have checked the old job, we run the unit test only the rest of the script fails, PowerShell does not report a failure so the build it is a success (but should not). In the new implementation, we run only the unit test until we adapt the system test. I have added the missing test for x-pack/winlogbeat

08:57:37 >> go test: Unit Test Passed
08:57:37 System testing x-pack/winlogbeat
08:57:38 go : warning: no packages being tested depend on matches for pattern github.com/elastic/beats/x-pack/winlogbeat/include
08:57:38 At C:\Users\jenkins\workspace\elastic+beats+master+multijob-windows\beat\x-pack\winlogbeat\label\windows\src\github.com
08:57:38 \elastic\beats\dev-tools\jenkins_ci.ps1:64 char:8
08:57:38 + exec { go test -race -c -cover -covermode=atomic -coverpkg $packages  ...
08:57:38 +        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
08:57:38     + CategoryInfo          : NotSpecified: (warning: no pac...logbeat/include:String) [], RemoteException
08:57:38     + FullyQualifiedErrorId : NativeCommandError

Jenkinsfile

Co-Authored-By: Andrew Kroh <[email protected]>

kuisathaverat · 2020-02-05T16:15:01Z

kuisathaverat · 2020-02-06T09:59:31Z

@andrewkroh if you are OK we can merge it and move to the next step

andrewkroh · 2020-02-06T23:58:36Z

Yeah, let's merge this in and I'll see if I can get someone to look at the beats that are failing. We can trigger this anywhere with the phrase used above, right?

The failing "Generators Beat Linux" might be an issue with the workspace having some root owned files from the previous test that cannot be cleaned up.

kuisathaverat · 2020-02-07T12:25:12Z

We can trigger this anywhere with the phrase used above, right?

I am afraid not until it listens for PRs

The failing "Generators Beat Linux" might be an issue with the workspace having some root owned files from the previous test that cannot be cleaned up.

yes, it is #16130

* [Filebeat] move create-[module,fileset,fields] to mage (#15836) - move create-[module,fileset,fields] to mage - make mage create commands available in x-pack/filebeat - change Makefile to use mage for create commands * Elasticsearch index must be lowercase (#16081) * Index names must be lowercase When indexing into Elasticsearch index names must always be lowercase. If the index or indices setting are configured to produce non-lowercase strings (e.g. by extracting part of the index name from the event contents), we need to normalize them to be lowercase. This change ensure that index names are always converted to lowercase. Static strings are converted to lowercase upfront, while dynamic strings will be post-processed. * update kafka/redis/LS output to guarantee lowercase index * add godoc * Regenerate expected files after changes in date parsing (#16139) Elasticsearch has modified the behaviour on date parsing when the date doesn't include timezone data. Regenerate a couple of golden files that are affected by this change. * Add autodiscover for aws_ec2 (#14823) * Add autodiscover for aws_ec2 * Add aws.ec2.* to autodiscover template * Fix a connection error in httpjson input (#16123) * Fix a connection error in httpjson input * Include document_id in decode_json_fields allowed fields (#16156) * ci: run test on Windows (#15570) * feat: run test on Windows * chore: parameter to enable/disable windows test * deleteDir before of the checkout * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Apply suggestions from code review * feat: apply dependency hierarchies * fit: use isChanged for all matches, and add the libbeat match where it is needed * feat: add x-pack/winlogbeat windows unit tests * fix: duplicate when condition * Update Jenkinsfile Co-Authored-By: Andrew Kroh <[email protected]> Co-authored-by: Andrew Kroh <[email protected]> * improve kubernetes.pod.cpu.usage.limit.pct field description (#16128) * upgrade github.com/gogo/protobuf/... to v1.3.1 (#16138) * [Filebeat] Add ECS tls & categorization fields to apache module (#16121) * Add ECS tls & categorization fields to apache module - tls.cipher (access) - tls.protocol (access) - tls.protocol_version (access) - event.kind (access) - event.category (access) - event.outcome (access) - lowercase http.request.method for ECS compliance (access) - event.kind (error) - event.category (error) - event.type (error) Closes #16032 * [Metricbeat] Add Overview dashboard to Tomcat module * [Metricbeat] Fix PostgreSQL Dashboard (#16132) * [Metricbeat] Fix PostgreSQL Dashboard * Update version * Fix: imports order (#16207) * [Metricbeat]kube-state-metrics: add storage class support (#16145) * add ksm storage class support * [Journalbeat] Improve parsing of syslog.pid in journalbeat to strip the username when present (#16116) * Improve parsing of syslog.pid in journalbeat to strip the username in pid when present. * Add entry to changelog with pull ID. * Improve the comment on the username strip. * [Agent] Allow CA cert pinning on the Elasticsearch output or any code that user tlscommon.TLSConfig builder. (#16019) * Add a sha256 pin for the CA Certificate When multiples CA are presents on the system we cannot ensure that a specific one was used to validates the chains exposer by the server. This PRs adds a `ca_sha256` option to the `tlscommon.TLSConfig` that is used by all the code that has to create a TCP client with TLS support. When the option is set, it will hook a new callback in the validation chains that will inspect the verified and validated chains by Go to ensure that a lets a certificate in the chains match the provided sha256. Usage example for the Elasticsearch output. ``` output.elasticsearch: hosts: [127.0.0.1:9200] ssl.ca_sha256: <base64_encoded_sha1> ``` You can generate the pin using the **openssl** binary with the following command: ``` openssl x509 -in ca.crt -pubkey -noout | openssl pkey -pubin -outform der | openssl dgst -sha256 -binary | openssl enc -base64 ``` OpenSSL's [documentation](https://www.openssl.org/docs/manmaster/man1/dgst.html) You will need to start Elasticsearch with the following options ```yaml xpack.security.enabled: true indices.id_field_data.enabled: true xpack.license.self_generated.type: trial xpack.security.http.ssl.enabled: true xpack.security.http.ssl.key: /etc/pki/localhost/localhost.key" xpack.security.http.ssl.certificate: /etc/pki/localhost/localhost.crt" xpack.security.http.ssl.certificate_authorities: /etc/pki/ca/ca.crt" ``` This pull request also include a new service in the docker-compose.yml that will start a new Elasticsearch server with TLS and security configured. * [docs] Add 7.6 breaking changes and release highlights (#16202) * [docs] Add early draft of Elastic Log Driver docs (#15799) * Index template will only be loaded if the configured output is Elasticsearch or Elastic Cloud (#16124) (#16225) Minor update to be more explicit on the index template loading requirement. Co-authored-by: romain-chanu <[email protected]> * Remove spaces in prometheus commented out option (#16233) * Fix: don't miss address scheme (#16205) * Fix: don't miss address scheme * Add unit test * Adjust source after code review * Add comment to method * Freeze virtualenv version until issue with CI is resolved (#16235) * [docs] Fix install command to match instructions on docker hub (#16249) * [docs] Add link to observability release blog (#16246) * ci(jenkins): enable fix-permissions to be executed without running make too (#16130) * ci(jenkins): enable fix-permissions to be executed without running make too * ci(jenkins): go modules are stored in the HOME path * ci(jenkins): fix permissions should run only if docker is enabled * Upgrade go-ucfg to version 0.8.2 (#16199) * Upgrade go-ucfg to master, for testing before 0.8.2 release. * Update notice. * Fix tests. * Update to the v0.8.2 release tag and remake NOTICE.txt. * Improve test name. * Add ingress nginx controller fileset (#16197) * update notice Co-authored-by: Lee Hinman <[email protected]> Co-authored-by: Steffen Siering <[email protected]> Co-authored-by: Jaime Soriano Pastor <[email protected]> Co-authored-by: kaiyan-sheng <[email protected]> Co-authored-by: Lei Qiu <[email protected]> Co-authored-by: Fae Charlton <[email protected]> Co-authored-by: Ivan Fernandez Calvo <[email protected]> Co-authored-by: Andrew Kroh <[email protected]> Co-authored-by: Chris Mark <[email protected]> Co-authored-by: Gil Raphaelli <[email protected]> Co-authored-by: Mario Castro <[email protected]> Co-authored-by: Dimitri Mazmanov <[email protected]> Co-authored-by: Marcin Tojek <[email protected]> Co-authored-by: Pablo Mercado <[email protected]> Co-authored-by: Blake Rouse <[email protected]> Co-authored-by: Pier-Hugues Pellerin <[email protected]> Co-authored-by: DeDe Morton <[email protected]> Co-authored-by: romain-chanu <[email protected]> Co-authored-by: Michal Pristas <[email protected]> Co-authored-by: Victor Martinez <[email protected]>

* feat: run test on Windows * chore: parameter to enable/disable windows test * deleteDir before of the checkout * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Apply suggestions from code review * feat: apply dependency hierarchies * fit: use isChanged for all matches, and add the libbeat match where it is needed * feat: add x-pack/winlogbeat windows unit tests * fix: duplicate when condition * Update Jenkinsfile Co-Authored-By: Andrew Kroh <[email protected]> Co-authored-by: Andrew Kroh <[email protected]>

* [Filebeat] move create-[module,fileset,fields] to mage (#15836) - move create-[module,fileset,fields] to mage - make mage create commands available in x-pack/filebeat - change Makefile to use mage for create commands * Elasticsearch index must be lowercase (#16081) * Index names must be lowercase When indexing into Elasticsearch index names must always be lowercase. If the index or indices setting are configured to produce non-lowercase strings (e.g. by extracting part of the index name from the event contents), we need to normalize them to be lowercase. This change ensure that index names are always converted to lowercase. Static strings are converted to lowercase upfront, while dynamic strings will be post-processed. * update kafka/redis/LS output to guarantee lowercase index * add godoc * Regenerate expected files after changes in date parsing (#16139) Elasticsearch has modified the behaviour on date parsing when the date doesn't include timezone data. Regenerate a couple of golden files that are affected by this change. * Add autodiscover for aws_ec2 (#14823) * Add autodiscover for aws_ec2 * Add aws.ec2.* to autodiscover template * Fix a connection error in httpjson input (#16123) * Fix a connection error in httpjson input * Include document_id in decode_json_fields allowed fields (#16156) * ci: run test on Windows (#15570) * feat: run test on Windows * chore: parameter to enable/disable windows test * deleteDir before of the checkout * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Apply suggestions from code review * feat: apply dependency hierarchies * fit: use isChanged for all matches, and add the libbeat match where it is needed * feat: add x-pack/winlogbeat windows unit tests * fix: duplicate when condition * Update Jenkinsfile Co-Authored-By: Andrew Kroh <[email protected]> Co-authored-by: Andrew Kroh <[email protected]> * improve kubernetes.pod.cpu.usage.limit.pct field description (#16128) * upgrade github.com/gogo/protobuf/... to v1.3.1 (#16138) * [Filebeat] Add ECS tls & categorization fields to apache module (#16121) * Add ECS tls & categorization fields to apache module - tls.cipher (access) - tls.protocol (access) - tls.protocol_version (access) - event.kind (access) - event.category (access) - event.outcome (access) - lowercase http.request.method for ECS compliance (access) - event.kind (error) - event.category (error) - event.type (error) Closes #16032 * [Metricbeat] Add Overview dashboard to Tomcat module * [Metricbeat] Fix PostgreSQL Dashboard (#16132) * [Metricbeat] Fix PostgreSQL Dashboard * Update version * Fix: imports order (#16207) * [Metricbeat]kube-state-metrics: add storage class support (#16145) * add ksm storage class support * [Journalbeat] Improve parsing of syslog.pid in journalbeat to strip the username when present (#16116) * Improve parsing of syslog.pid in journalbeat to strip the username in pid when present. * Add entry to changelog with pull ID. * Improve the comment on the username strip. * [Agent] Allow CA cert pinning on the Elasticsearch output or any code that user tlscommon.TLSConfig builder. (#16019) * Add a sha256 pin for the CA Certificate When multiples CA are presents on the system we cannot ensure that a specific one was used to validates the chains exposer by the server. This PRs adds a `ca_sha256` option to the `tlscommon.TLSConfig` that is used by all the code that has to create a TCP client with TLS support. When the option is set, it will hook a new callback in the validation chains that will inspect the verified and validated chains by Go to ensure that a lets a certificate in the chains match the provided sha256. Usage example for the Elasticsearch output. ``` output.elasticsearch: hosts: [127.0.0.1:9200] ssl.ca_sha256: <base64_encoded_sha1> ``` You can generate the pin using the **openssl** binary with the following command: ``` openssl x509 -in ca.crt -pubkey -noout | openssl pkey -pubin -outform der | openssl dgst -sha256 -binary | openssl enc -base64 ``` OpenSSL's [documentation](https://www.openssl.org/docs/manmaster/man1/dgst.html) You will need to start Elasticsearch with the following options ```yaml xpack.security.enabled: true indices.id_field_data.enabled: true xpack.license.self_generated.type: trial xpack.security.http.ssl.enabled: true xpack.security.http.ssl.key: /etc/pki/localhost/localhost.key" xpack.security.http.ssl.certificate: /etc/pki/localhost/localhost.crt" xpack.security.http.ssl.certificate_authorities: /etc/pki/ca/ca.crt" ``` This pull request also include a new service in the docker-compose.yml that will start a new Elasticsearch server with TLS and security configured. * [docs] Add 7.6 breaking changes and release highlights (#16202) * [docs] Add early draft of Elastic Log Driver docs (#15799) * Index template will only be loaded if the configured output is Elasticsearch or Elastic Cloud (#16124) (#16225) Minor update to be more explicit on the index template loading requirement. Co-authored-by: romain-chanu <[email protected]> * Remove spaces in prometheus commented out option (#16233) * Fix: don't miss address scheme (#16205) * Fix: don't miss address scheme * Add unit test * Adjust source after code review * Add comment to method * Freeze virtualenv version until issue with CI is resolved (#16235) * [docs] Fix install command to match instructions on docker hub (#16249) * [docs] Add link to observability release blog (#16246) * ci(jenkins): enable fix-permissions to be executed without running make too (#16130) * ci(jenkins): enable fix-permissions to be executed without running make too * ci(jenkins): go modules are stored in the HOME path * ci(jenkins): fix permissions should run only if docker is enabled * Upgrade go-ucfg to version 0.8.2 (#16199) * Upgrade go-ucfg to master, for testing before 0.8.2 release. * Update notice. * Fix tests. * Update to the v0.8.2 release tag and remake NOTICE.txt. * Improve test name. * Add ingress nginx controller fileset (#16197) * update notice Co-authored-by: Lee Hinman <[email protected]> Co-authored-by: Steffen Siering <[email protected]> Co-authored-by: Jaime Soriano Pastor <[email protected]> Co-authored-by: kaiyan-sheng <[email protected]> Co-authored-by: Lei Qiu <[email protected]> Co-authored-by: Fae Charlton <[email protected]> Co-authored-by: Ivan Fernandez Calvo <[email protected]> Co-authored-by: Andrew Kroh <[email protected]> Co-authored-by: Chris Mark <[email protected]> Co-authored-by: Gil Raphaelli <[email protected]> Co-authored-by: Mario Castro <[email protected]> Co-authored-by: Dimitri Mazmanov <[email protected]> Co-authored-by: Marcin Tojek <[email protected]> Co-authored-by: Pablo Mercado <[email protected]> Co-authored-by: Blake Rouse <[email protected]> Co-authored-by: Pier-Hugues Pellerin <[email protected]> Co-authored-by: DeDe Morton <[email protected]> Co-authored-by: romain-chanu <[email protected]> Co-authored-by: Michal Pristas <[email protected]> Co-authored-by: Victor Martinez <[email protected]>

* feat: run test on Windows * chore: parameter to enable/disable windows test * deleteDir before of the checkout * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Apply suggestions from code review * feat: apply dependency hierarchies * fit: use isChanged for all matches, and add the libbeat match where it is needed * feat: add x-pack/winlogbeat windows unit tests * fix: duplicate when condition * Update Jenkinsfile Co-Authored-By: Andrew Kroh <[email protected]> Co-authored-by: Andrew Kroh <[email protected]>

* cherry-pick aad07ad * Add stages to Jenkins pipeline (#14484) * ci: avoid to modify go.mod file (#16842) * Ensure install scripts only install if needed (#20349) * ci: fix warnings with wildcards and archive system-tests (#18695) * ci: run test on Windows (#15570) * [CI] fail if not possible to install python3 (#19164) * [CI] lint stage doesn't produce test reports (#21888) * [CI] Add stage name in the step (#21887) * [CI] kind setup fails sometimes (#21857) * [CI] refactor runbld post build action (#21256) * [CI] Archive build reasons (#21347) * [CI] Fix runbld when workspace does not exist (#21350) * [CI] Send slack message with build status (#21428) * [CI] Setup git config globally (#21562) * [CI] fix 'no matches found within 10000' (#21466) * [CI] Change notification channel (#21559) * [CI] Support skip-ci label (#21377) * [BUILD][CI] fetch dependencies with retry (#21614) * Disable x-pack/metricbeat in linux since build is broken #21962 * Disable tests for windows in auditbeat. See #21958 * Disable integration testing for libbeat. See #21959 * Disable integration testing for metricbeat. See #21961 * Disable windows build for auditbeat. See #21958 * Disable build for libbeat. See #21959 * Disable build for metricbeat. See #21961 * Disable windows build for metricbeat. See #22257 * Disable build for x-pack/metricbeat. See #21962 * Disable windows build for x-pack/winlogbeat. See #22263 * Disable windows build for x-pack/filebeat. See #21963 * integration-tests-environment is not available for winlogbeat. see #22265 * integration-tests-environment is not available for packetbeat. see #22266 * Build not available for x-pack/metricbeats in 6.8. See #22269 * Cherry-pick 96c60d0

* [Filebeat] move create-[module,fileset,fields] to mage (elastic#15836) - move create-[module,fileset,fields] to mage - make mage create commands available in x-pack/filebeat - change Makefile to use mage for create commands * Elasticsearch index must be lowercase (elastic#16081) * Index names must be lowercase When indexing into Elasticsearch index names must always be lowercase. If the index or indices setting are configured to produce non-lowercase strings (e.g. by extracting part of the index name from the event contents), we need to normalize them to be lowercase. This change ensure that index names are always converted to lowercase. Static strings are converted to lowercase upfront, while dynamic strings will be post-processed. * update kafka/redis/LS output to guarantee lowercase index * add godoc * Regenerate expected files after changes in date parsing (elastic#16139) Elasticsearch has modified the behaviour on date parsing when the date doesn't include timezone data. Regenerate a couple of golden files that are affected by this change. * Add autodiscover for aws_ec2 (elastic#14823) * Add autodiscover for aws_ec2 * Add aws.ec2.* to autodiscover template * Fix a connection error in httpjson input (elastic#16123) * Fix a connection error in httpjson input * Include document_id in decode_json_fields allowed fields (elastic#16156) * ci: run test on Windows (elastic#15570) * feat: run test on Windows * chore: parameter to enable/disable windows test * deleteDir before of the checkout * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Update Jenkinsfile * Apply suggestions from code review * feat: apply dependency hierarchies * fit: use isChanged for all matches, and add the libbeat match where it is needed * feat: add x-pack/winlogbeat windows unit tests * fix: duplicate when condition * Update Jenkinsfile Co-Authored-By: Andrew Kroh <[email protected]> Co-authored-by: Andrew Kroh <[email protected]> * improve kubernetes.pod.cpu.usage.limit.pct field description (elastic#16128) * upgrade github.com/gogo/protobuf/... to v1.3.1 (elastic#16138) * [Filebeat] Add ECS tls & categorization fields to apache module (elastic#16121) * Add ECS tls & categorization fields to apache module - tls.cipher (access) - tls.protocol (access) - tls.protocol_version (access) - event.kind (access) - event.category (access) - event.outcome (access) - lowercase http.request.method for ECS compliance (access) - event.kind (error) - event.category (error) - event.type (error) Closes elastic#16032 * [Metricbeat] Add Overview dashboard to Tomcat module * [Metricbeat] Fix PostgreSQL Dashboard (elastic#16132) * [Metricbeat] Fix PostgreSQL Dashboard * Update version * Fix: imports order (elastic#16207) * [Metricbeat]kube-state-metrics: add storage class support (elastic#16145) * add ksm storage class support * [Journalbeat] Improve parsing of syslog.pid in journalbeat to strip the username when present (elastic#16116) * Improve parsing of syslog.pid in journalbeat to strip the username in pid when present. * Add entry to changelog with pull ID. * Improve the comment on the username strip. * [Agent] Allow CA cert pinning on the Elasticsearch output or any code that user tlscommon.TLSConfig builder. (elastic#16019) * Add a sha256 pin for the CA Certificate When multiples CA are presents on the system we cannot ensure that a specific one was used to validates the chains exposer by the server. This PRs adds a `ca_sha256` option to the `tlscommon.TLSConfig` that is used by all the code that has to create a TCP client with TLS support. When the option is set, it will hook a new callback in the validation chains that will inspect the verified and validated chains by Go to ensure that a lets a certificate in the chains match the provided sha256. Usage example for the Elasticsearch output. ``` output.elasticsearch: hosts: [127.0.0.1:9200] ssl.ca_sha256: <base64_encoded_sha1> ``` You can generate the pin using the **openssl** binary with the following command: ``` openssl x509 -in ca.crt -pubkey -noout | openssl pkey -pubin -outform der | openssl dgst -sha256 -binary | openssl enc -base64 ``` OpenSSL's [documentation](https://www.openssl.org/docs/manmaster/man1/dgst.html) You will need to start Elasticsearch with the following options ```yaml xpack.security.enabled: true indices.id_field_data.enabled: true xpack.license.self_generated.type: trial xpack.security.http.ssl.enabled: true xpack.security.http.ssl.key: /etc/pki/localhost/localhost.key" xpack.security.http.ssl.certificate: /etc/pki/localhost/localhost.crt" xpack.security.http.ssl.certificate_authorities: /etc/pki/ca/ca.crt" ``` This pull request also include a new service in the docker-compose.yml that will start a new Elasticsearch server with TLS and security configured. * [docs] Add 7.6 breaking changes and release highlights (elastic#16202) * [docs] Add early draft of Elastic Log Driver docs (elastic#15799) * Index template will only be loaded if the configured output is Elasticsearch or Elastic Cloud (elastic#16124) (elastic#16225) Minor update to be more explicit on the index template loading requirement. Co-authored-by: romain-chanu <[email protected]> * Remove spaces in prometheus commented out option (elastic#16233) * Fix: don't miss address scheme (elastic#16205) * Fix: don't miss address scheme * Add unit test * Adjust source after code review * Add comment to method * Freeze virtualenv version until issue with CI is resolved (elastic#16235) * [docs] Fix install command to match instructions on docker hub (elastic#16249) * [docs] Add link to observability release blog (elastic#16246) * ci(jenkins): enable fix-permissions to be executed without running make too (elastic#16130) * ci(jenkins): enable fix-permissions to be executed without running make too * ci(jenkins): go modules are stored in the HOME path * ci(jenkins): fix permissions should run only if docker is enabled * Upgrade go-ucfg to version 0.8.2 (elastic#16199) * Upgrade go-ucfg to master, for testing before 0.8.2 release. * Update notice. * Fix tests. * Update to the v0.8.2 release tag and remake NOTICE.txt. * Improve test name. * Add ingress nginx controller fileset (elastic#16197) * update notice Co-authored-by: Lee Hinman <[email protected]> Co-authored-by: Steffen Siering <[email protected]> Co-authored-by: Jaime Soriano Pastor <[email protected]> Co-authored-by: kaiyan-sheng <[email protected]> Co-authored-by: Lei Qiu <[email protected]> Co-authored-by: Fae Charlton <[email protected]> Co-authored-by: Ivan Fernandez Calvo <[email protected]> Co-authored-by: Andrew Kroh <[email protected]> Co-authored-by: Chris Mark <[email protected]> Co-authored-by: Gil Raphaelli <[email protected]> Co-authored-by: Mario Castro <[email protected]> Co-authored-by: Dimitri Mazmanov <[email protected]> Co-authored-by: Marcin Tojek <[email protected]> Co-authored-by: Pablo Mercado <[email protected]> Co-authored-by: Blake Rouse <[email protected]> Co-authored-by: Pier-Hugues Pellerin <[email protected]> Co-authored-by: DeDe Morton <[email protected]> Co-authored-by: romain-chanu <[email protected]> Co-authored-by: Michal Pristas <[email protected]> Co-authored-by: Victor Martinez <[email protected]>

kuisathaverat added automation ci labels Jan 15, 2020

kuisathaverat self-assigned this Jan 15, 2020

kuisathaverat force-pushed the win-test branch 5 times, most recently from 61ebe3f to 41b7173 Compare January 15, 2020 15:25

kuisathaverat force-pushed the win-test branch 5 times, most recently from 5864288 to 0ba06f5 Compare January 15, 2020 18:12

feat: run test on Windows

25dd238

kuisathaverat force-pushed the win-test branch from 0ba06f5 to 25dd238 Compare January 16, 2020 09:29

kuisathaverat and others added 3 commits January 16, 2020 16:21

chore: parameter to enable/disable windows test

b38031d

deleteDir before of the checkout

914d113

Merge branch 'master' into win-test

255a38a

kuisathaverat added 3 commits January 22, 2020 15:29

Update Jenkinsfile

f3ea4eb

Update Jenkinsfile

85bd542

Update Jenkinsfile

c9546e6

andrewkroh reviewed Jan 29, 2020

View reviewed changes

kuisathaverat added 2 commits January 29, 2020 11:36

Update Jenkinsfile

de66312

Update Jenkinsfile

9b8d8ca

kuisathaverat commented Jan 29, 2020

View reviewed changes

Jenkinsfile Outdated Show resolved Hide resolved

kuisathaverat commented Jan 29, 2020

View reviewed changes

Jenkinsfile Outdated Show resolved Hide resolved

kuisathaverat commented Jan 29, 2020

View reviewed changes

Jenkinsfile Outdated Show resolved Hide resolved

andrewkroh approved these changes Feb 5, 2020

View reviewed changes

Jenkinsfile Show resolved Hide resolved

fit: use isChanged for all matches, and add the libbeat match where i…

2e208aa

…t is needed

andrewkroh approved these changes Feb 5, 2020

View reviewed changes

feat: add x-pack/winlogbeat windows unit tests

6e6eeb6

kuisathaverat changed the title ~~[WIP] ci: run test on Windows~~ ci: run test on Windows Feb 5, 2020

fix: duplicate when condition

1642570

andrewkroh reviewed Feb 5, 2020

View reviewed changes

Jenkinsfile Outdated Show resolved Hide resolved

Update Jenkinsfile

f31c40d

Co-Authored-By: Andrew Kroh <[email protected]>

kuisathaverat merged commit 1324279 into elastic:master Feb 7, 2020

zube bot added [zube]: Done and removed [zube]: In Review labels Feb 7, 2020

alvarolobato removed the [zube]: Done label Feb 19, 2020

kvch mentioned this pull request Apr 2, 2020

Cherry-pick #17388 to 7.x: Fix command which installs mage #17436

Merged

1 task

kuisathaverat deleted the win-test branch December 24, 2020 11:05

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

ci: run test on Windows #15570

ci: run test on Windows #15570

kuisathaverat commented Jan 15, 2020 •

edited

Loading

kuisathaverat commented Jan 15, 2020

kuisathaverat commented Jan 16, 2020

kuisathaverat commented Jan 22, 2020

andrewkroh left a comment

andrewkroh left a comment

kuisathaverat commented Feb 5, 2020

kuisathaverat commented Feb 5, 2020

kuisathaverat commented Feb 6, 2020

andrewkroh commented Feb 6, 2020

kuisathaverat commented Feb 7, 2020

ci: run test on Windows #15570

ci: run test on Windows #15570

Conversation

kuisathaverat commented Jan 15, 2020 • edited Loading

kuisathaverat commented Jan 15, 2020

kuisathaverat commented Jan 16, 2020

kuisathaverat commented Jan 22, 2020

andrewkroh left a comment

Choose a reason for hiding this comment

andrewkroh left a comment

Choose a reason for hiding this comment

kuisathaverat commented Feb 5, 2020

kuisathaverat commented Feb 5, 2020

kuisathaverat commented Feb 6, 2020

andrewkroh commented Feb 6, 2020

kuisathaverat commented Feb 7, 2020

kuisathaverat commented Jan 15, 2020 •

edited

Loading