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

ci: set a version for Chocolatey #7588

Merged
merged 6 commits into from
Jun 19, 2023
Merged

Conversation

celalettin1286
Copy link
Contributor

@celalettin1286 celalettin1286 commented Jun 19, 2023

Fixes #7574 setting Chocolatey version to 1.4.0 since the version doesn't require .NET 4.8, also ensures the host version matches the container version.


Enter [N/A] in the box, if an item is not applicable to your change.

Testing
Before we can approve your change; please submit the following in a comment:

  • Example configuration file for the change
  • Debug log output from testing the change
  • Attached Valgrind output that shows no leaks or memory corruption was found

If this is a change to packaging of containers or native binaries then please confirm it works for all targets.

  • Run local packaging test showing all targets (including any new ones) build.
  • Set ok-package-test label to test for all targets (requires maintainer to do).

Documentation

  • Documentation required for this feature

Backporting

  • Backport to latest stable release.

Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.

fixes #7574 setting Chocolatey version to 1.4.0 since the version doesn't require .NET 4.8

Signed-off-by: Celalettin Calis <[email protected]>
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 12:21 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 12:21 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 12:21 — with GitHub Actions Inactive
@patrick-stephens patrick-stephens added the ok-package-test Run PR packaging tests label Jun 19, 2023
@patrick-stephens
Copy link
Contributor

I've triggered the full build to see if it resolves things.

@patrick-stephens
Copy link
Contributor

@celalettin1286 please update the https://github.com/fluent/fluent-bit/blob/master/.github/workflows/pr-image-tests.yaml workflow so it builds the Windows containers too - that will then verify your change as well.

@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 12:44 — with GitHub Actions Inactive
@patrick-stephens patrick-stephens added Windows Bugs and requests about Windows platforms packaging and removed docs-required labels Jun 19, 2023
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 13:31 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 13:31 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 13:31 — with GitHub Actions Inactive
Signed-off-by: celalettin1286 <[email protected]>
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 13:45 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 13:45 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 13:45 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 13:57 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 13:57 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 13:57 — with GitHub Actions Inactive
@patrick-stephens patrick-stephens removed the ok-package-test Run PR packaging tests label Jun 19, 2023
.github/workflows/pr-image-tests.yaml Outdated Show resolved Hide resolved
.github/workflows/pr-image-tests.yaml Outdated Show resolved Hide resolved
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 14:23 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 14:56 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 14:56 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 14:56 — with GitHub Actions Inactive
Signed-off-by: celalettin1286 <[email protected]>
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 14:59 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 14:59 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 14:59 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 temporarily deployed to pr June 19, 2023 15:21 — with GitHub Actions Inactive
@celalettin1286 celalettin1286 merged commit 41d3e17 into master Jun 19, 2023
@celalettin1286 celalettin1286 deleted the ci-chocolatey-version branch June 19, 2023 15:51
celalettin1286 added a commit that referenced this pull request Jun 21, 2023
* ci: set a version for Chocolatey

fixes #7574 setting Chocolatey version to 1.4.0 since the version doesn't require .NET 4.8

Signed-off-by: Celalettin Calis <[email protected]>

* ci: added windows tests to pr image tests

Signed-off-by: celalettin1286 <[email protected]>

* ci: changed build method

Signed-off-by: celalettin1286 <[email protected]>

* ci: changed runner to windows 2019

Signed-off-by: celalettin1286 <[email protected]>

* ci: add matrix for windows 2019 and 2022

Signed-off-by: celalettin1286 <[email protected]>

* change job title

Signed-off-by: celalettin1286 <[email protected]>

---------

Signed-off-by: Celalettin Calis <[email protected]>
Signed-off-by: celalettin1286 <[email protected]>
celalettin1286 added a commit that referenced this pull request Jun 21, 2023
* ci: set a version for Chocolatey

fixes #7574 setting Chocolatey version to 1.4.0 since the version doesn't require .NET 4.8



* ci: added windows tests to pr image tests



* ci: changed build method



* ci: changed runner to windows 2019



* ci: add matrix for windows 2019 and 2022



* change job title



---------

Signed-off-by: Celalettin Calis <[email protected]>
Signed-off-by: celalettin1286 <[email protected]>
leonardo-albertovich pushed a commit that referenced this pull request Jul 17, 2023
* ci: set a version for Chocolatey

fixes #7574 setting Chocolatey version to 1.4.0 since the version doesn't require .NET 4.8

Signed-off-by: Celalettin Calis <[email protected]>

* ci: added windows tests to pr image tests

Signed-off-by: celalettin1286 <[email protected]>

* ci: changed build method

Signed-off-by: celalettin1286 <[email protected]>

* ci: changed runner to windows 2019

Signed-off-by: celalettin1286 <[email protected]>

* ci: add matrix for windows 2019 and 2022

Signed-off-by: celalettin1286 <[email protected]>

* change job title

Signed-off-by: celalettin1286 <[email protected]>

---------

Signed-off-by: Celalettin Calis <[email protected]>
Signed-off-by: celalettin1286 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packaging Windows Bugs and requests about Windows platforms
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Windows containers failing to build in CI
2 participants