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

examples: filter_wasm_go: Upgrade to support Go 1.22.5 and TinyGo 0.32 #9100

Closed
wants to merge 2 commits into from

Conversation

opencmit2
Copy link
Contributor

@opencmit2 opencmit2 commented Jul 17, 2024

examples: filter_wasm_go: Upgrade to support Go 1.22.5 and TinyGo 0.32

Changes include:

  1. Go Toolchain Update: The Go toolchain used for building WebAssembly modules has
    been upgraded to Go 1.22.5. This ensures that the compiled WASM modules take advantage
    of the latest compiler optimizations and language improvements.

  2. TinyGo Integration: TinyGo support has been updated to version 0.32, which
    introduces new optimizations and potentially new APIs for WebAssembly development.
    This allows for smaller binary sizes and potentially better performance in
    resource-constrained environments.

  3. Documentation Updates: The documentation for filter_wasm_go has been updated to
    reflect the new supported versions of Go and TinyGo, as well as any changes in
    build/deployment instructions.

  4. Dependency Management: Any third-party dependencies required for building or
    testing the WebAssembly filters have been updated to be compatible with the new
    versions of Go and TinyGo.


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.

@cosmo0920
Copy link
Contributor

@opencmit2 Could you use examples: filter_wasm_go: Upgrade to support Go 1.22.5 and TinyGo 0.32 in your commit message to follow the contribution guide?

@opencmit2 opencmit2 changed the title [Fluent Bit / filter_wasm_go] Upgrade to support Go 1.22.5 and TinyGo 0.32 examples: filter_wasm_go: Upgrade to support Go 1.22.5 and TinyGo 0.32 Jul 18, 2024
@opencmit2
Copy link
Contributor Author

@cosmo0920 Is this the case?

@cosmo0920
Copy link
Contributor

@cosmo0920 Is this the case?

No. You should update your commit message with: examples: filter_wasm_go: Upgrade to support Go 1.22.5 and TinyGo 0.32.
I didn't observed that yet.

@opencmit2
Copy link
Contributor Author

@cosmo0920 Modified, but I accidentally used a private account to submit

Copy link

@im-jinxinwang im-jinxinwang left a comment

Choose a reason for hiding this comment

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

examples: filter_wasm_go: Upgrade to support Go 1.22.5 and TinyGo 0.32

@opencmit2 opencmit2 closed this Jul 18, 2024
@opencmit2 opencmit2 deleted the edit_wasm_go branch July 18, 2024 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants