Commit Graph

86 Commits

Author SHA1 Message Date
Jens Geyer
2f214c28d8 THRIFT-5902 Add net10 support
Client: netstd
Patch: Jens Geyer
2025-11-14 21:17:28 +01:00
Carel
ed55a18c30 Remove kotlin cross tests (#3236)
- They have been failing for months now without any actions
- The red builds have already desensitized the project to checking for failures
2025-11-14 16:54:52 +08:00
Carel
e720e6fecb THRIFT-5899: Python tests fail for the Appveyor MSVC builds (#3229)
* Temporarily switch off shared build

- See THRIFT-5898

* Skip type_hints tests for python lower than 3.7

- PR#2929 called out that the changes breaks Python 3.5 since types came in in 3.6
- Python 3.6 errors out with 'from __future__ import annotations' since it looks like it was only added in 3.7

* More appveyer issues on Windows due to old python

* Remove enum tests for old python versions

- Getting `raise TApplicationException(TApplicationException.MISSING_RESULT, "testEnum failed: unknown result"` error
- PR#2825 state it is a breaking change, not sure why and for what version of Python

* Disable SSL tests for old Python

- Appveyor error: ` AttributeError: module 'ssl' has no attribute 'PROTOCOL_TLS_CLIENT'`

* Can't get the test to skip so revert the change that broke it

- See PR#3050

* THRIFT-5900: Pin the cross test stage to python 3.13

- See https://issues.apache.org/jira/browse/THRIFT-5900
2025-11-07 08:48:27 +00:00
dependabot[bot]
1a5b7fc85b Bump actions/setup-python from 5 to 6
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 5 to 6.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](https://github.com/actions/setup-python/compare/v5...v6)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2025-10-01 07:34:18 -07:00
dependabot[bot]
755f7d9faf Bump actions/setup-go from 5 to 6
Bumps [actions/setup-go](https://github.com/actions/setup-go) from 5 to 6.
- [Release notes](https://github.com/actions/setup-go/releases)
- [Commits](https://github.com/actions/setup-go/compare/v5...v6)

---
updated-dependencies:
- dependency-name: actions/setup-go
  dependency-version: '6'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2025-10-01 07:33:25 -07:00
dependabot[bot]
eaec898df1 Bump actions/setup-java from 4 to 5 (#3206)
Bumps [actions/setup-java](https://github.com/actions/setup-java) from 4 to 5.
- [Release notes](https://github.com/actions/setup-java/releases)
- [Commits](https://github.com/actions/setup-java/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-java
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2025-09-05 15:53:08 -04:00
dependabot[bot]
55783f90db Bump actions/download-artifact from 4 to 5 (#3207)
Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 4 to 5.
- [Release notes](https://github.com/actions/download-artifact/releases)
- [Commits](https://github.com/actions/download-artifact/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/download-artifact
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2025-09-05 15:52:26 -04:00
dependabot[bot]
e6f532421c Bump actions/checkout from 4 to 5 (#3208)
Bumps [actions/checkout](https://github.com/actions/checkout) from 4 to 5.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](https://github.com/actions/checkout/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-version: '5'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
2025-09-05 15:50:09 -04:00
Carel Combrink
fbe685aae1 Add cpp cross tests to the build workflow
- Remove usage of v0.16 thrift files for C++ since UUID support was added
- Need to install the locals for some of the unit tests
- Fix UUID support for THeaderProtocol
    - Without this the protocol went into an infinite loop due to virtual function calls that recursed to itself
    - Best case was a crash, worst case was process got stuck
- Fix UUID support for TProtocolTap
- Sorted the known failures
- Mark cpp and java ssl tests as known failures
2025-08-28 00:36:39 +02:00
Yuxuan 'fishy' Wang
3b86252e05 Update supported go versions to 1.24+1.25
Drop support for go 1.23 as it's no longer supported by upstream.
2025-08-13 13:32:53 -07:00
Yuxuan 'fishy' Wang
716835fa76 Fix cross-test
1. Comment in the middle of multi-line shell command joined by \ does
   not work, change to multiple individual commands instead.
2. Fix a py 2vs3 compat issue missed in 24df0a5 used in cross runner.
2025-05-29 08:42:25 -07:00
Jens Geyer
63b7a263fc THRIFT-5864 swift CI broken & no maintainer around to fix
Client: swift
Patch: Jens Geyer
2025-05-25 15:46:27 +02:00
Kino Roy
29d8773e7c THRIFT-4838: Add unix socket support for Swift
Client: Swift
Patch: Kino Roy
2025-04-28 23:42:16 +02:00
Jens Geyer
1834bfc957 added -y switch to prevent hang during CI run 2025-04-18 11:22:29 +02:00
Yuxuan 'fishy' Wang
7e45f58ad7 Remove ubuntu-20.04 from github actions
This is fully unsupported [1], and prevents further tests from running.

[1]: https://github.com/actions/runner-images/issues/11101
2025-04-17 09:02:03 -07:00
Cameron Martin
a675c4f858 THRIFT-5564: Add nodets tests to Github Actions
Client: nodets

This adds the tests for the "nodets" library. This isn't an actual library, and are just another set of tests for the nodejs library, but also using typescript. Therefore I have added them in the existing "nodejs" job.
2025-02-13 22:15:26 +01:00
Yuxuan 'fishy' Wang
624118f61f Update supported go versions
With the release of go 1.24.0, go 1.22.x is no longer supported. Update
supported go versions to go 1.23 and go 1.24.
2025-02-11 15:11:10 -08:00
Jens Geyer
8a5538a9a1 net9 on Noble only via NET backports 2025-02-11 23:16:46 +01:00
Cameron Martin
da54fc814c THRIFT-5819: use latest rustc version for rustlib (#3085)
Client: rust

This upgrades the version of rust in the rust-toolchain file, docs and dockerfiles. Doing so requires a few changes to the source, mainly to fix or silence new warnings.

Submitted on behalf of a third-party: Jiayu Liu
Derived from the following PR: https://github.com/apache/thrift/pull/3045

Co-authored-by: Jiayu Liu <jiayu@hey.com>
2025-01-12 16:55:45 +08:00
Cameron Martin
ab70652576 THRIFT-5564: Add nodejs tests to github actions (#3082)
These tests exist, but don't currently run on github actions. This adds a new job to run these.

This also fixes the regression in the tests caused by https://github.com/apache/thrift/pull/3014.
2025-01-12 16:54:49 +08:00
Jens Geyer
3a37d15341 net9 sdk is NOT YET installed by default 2024-11-22 23:25:49 +01:00
Jens Geyer
e26b4a8384 THRIFT-5832 Drop net6 support and add net9 instead
Client: netstd
Patch: Jens Geyer
2024-11-22 23:25:49 +01:00
Jens Geyer
a085b0ecee Revert "GitHub cmake workflow to build libraries as well [skip ci] (#2679)"
This reverts commit b65ec607db.
2024-11-14 23:16:37 +01:00
Jiayu Liu
b65ec607db GitHub cmake workflow to build libraries as well [skip ci] (#2679)
* cmake to build with java and kotlin

* add kotlin

* turn off python in cmake

* cmake and cpack

---------

Co-authored-by: Jiayu Liu <jiayu.liu@airbnb.com>
2024-11-06 11:39:01 +08:00
Jiayu Liu
e7f1a26a95 use 24.04 for github ci jobs (#3044) 2024-11-06 11:37:25 +08:00
Alexandre Detiste
8f9fba8a94 remove dependency on Six 2024-11-01 18:49:00 +01:00
Xuanwo
a32990df27 ci(rust): Add release workflow for rust, Update release_rust.yml
This closes #3027

Signed-off-by: Xuanwo <github@xuanwo.io>
2024-09-07 13:01:57 +02:00
Sven Roederer
cb9ceada55 GHA/build: run tests for netstd
Build and test the netstd library so there is some basic testing.
Run tests on ubuntu-22.04, as 20.04 wil be EoL soon and also had some issues with dotnet-sdk.
2024-08-22 13:39:31 +02:00
Yuxuan 'fishy' Wang
cf0b803512 Fix python build on Github Actions
There were some recent changes causing six to be installed on the `make
install` step instead of (implicitly) on the `Python setup` step before,
and the six installed on `make install` step was not available to the
next `make check` step and causing errors.

Install six on `Python setup` step explicitly instead.
2024-08-20 17:30:25 -07:00
Yuxuan 'fishy' Wang
91565d490e Update supported go versions
Client: go

With the release of go 1.23, update supported go versions to 1.22+1.23
according to our go support policy.

Also update the code to use the new range loop feature introduced in go
1.22 when appropriate.

Also fix a bug in TSSLServerSocket.Addr that it does not return the
listener address.
2024-08-14 10:39:41 -07:00
Sven Roederer
3d02c0f94e GHA/build: bring lib-php job to same versions of artifact-action and ubuntu-image 2024-07-25 11:59:25 +02:00
Sven Roederer
502a46a737 GHA: switch to ubuntu-22.04 as primary image
Ubuntu-20.04 will be EoL in 9 months, therefore let's use the next LTS version as default.
Images for ubuntu-24.04 are available, so use them also for building the compiler.
2024-07-25 11:59:25 +02:00
Sven Roederer
18b0de6d65 GHA/build: upgrade artifact-actions from v3 to v4
V3 is based on Node.js v16 and will get deprecated. Currently it's just causing a warning.

With v4 the artifact-upload action fails as all matrix-jobs use the same name and the
default prevents overwriting existing ones. Therefore we use individual artifact names
which will also preserve all logs, not just a singe one only.
2024-07-25 11:59:25 +02:00
Volodymyr Panivko
68139d117f [THRIFT-5757] Unit tests for php lib
Client: php
Patch: Volodymyr Panivko

This closes #2951
2024-04-07 22:52:47 +02:00
Volodymyr Panivko
99130046c4 [THRIFT-5757] Unit tests for php lib 2024-03-12 19:02:50 +01:00
Volodymyr Panivko
14fc2be20c [THRIFT-5757] Unit tests for php lib 2024-02-25 22:21:42 +01:00
Volodymyr Panivko
ac52d8dac1 [THRIFT-5758] PHP 8.2 Deprecate dynamic properties 2024-02-25 12:22:46 +01:00
vladimir.panivko
f6927020dd [THRIFT-5760] Update minimal version of php 2024-02-25 12:21:04 +01:00
Yuxuan 'fishy' Wang
3b1c30c5a7 Add license header to pypi workflow file 2024-02-21 23:36:04 +01:00
Volodymyr Panivko
8e828c03d4 THRIFT-5756 Run php tests in github actions 2024-02-21 09:10:28 +01:00
Yuxuan 'fishy' Wang
84b35ff828 THRIFT-5688: Add PyPI publishing github actions
This is tested with https://github.com/apache/thrift/pull/2927, which
published to https://test.pypi.org/project/thrift-test/. I tested
locally with:

    (venv) fishy@penguin:~/work/test$ pip install -i https://test.pypi.org/simple/ thrift-test
    Looking in indexes: https://test.pypi.org/simple/
    Collecting thrift-test
      Downloading 5885ea1406/thrift-test-0.21.0.tar.gz (62 kB)
         ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 62.3/62.3 kB 1.4 MB/s eta 0:00:00
      Preparing metadata (setup.py) ... done
    Collecting six>=1.7.2 (from thrift-test)
      Downloading 238e259082/six-1.10.0.tar.gz (29 kB)
      Preparing metadata (setup.py) ... done
    Building wheels for collected packages: thrift-test, six
      Building wheel for thrift-test (setup.py) ... done
      Created wheel for thrift-test: filename=thrift_test-0.21.0-cp311-cp311-linux_x86_64.whl size=416914 sha256=3a972bc562be7ed19cb37399e444ed0d373cde5319023974080b625e550901d4
      Stored in directory: /home/fishy/.cache/pip/wheels/45/20/1f/d3e1b869ac068d63ca2b2c13a2f4e33a80b360fae7091c8a9b
      Building wheel for six (setup.py) ... done
      Created wheel for six: filename=six-1.10.0-py2.py3-none-any.whl size=9942 sha256=74014380446ccf331366316cec0b1aaf40e0162e70307622b493e38e8451115f
      Stored in directory: /home/fishy/.cache/pip/wheels/e4/18/d0/e02474c90dcf14c511c0f52145d7e72e41ff3fb80b330ba58e
    Successfully built thrift-test six
    Installing collected packages: six, thrift-test
    Successfully installed six-1.10.0 thrift-test-0.21.0
    (venv) fishy@penguin:~/work/test$ python3
    Python 3.11.7 (main, Dec  8 2023, 14:22:46) [GCC 13.2.0] on linux
    Type "help", "copyright", "credits" or "license" for more information.
    >>> from thrift.transport import TSocket
    >>> transport = TSocket.TSocket('localhost', 9090)
    >>> transport
    <thrift.transport.TSocket.TSocket object at 0x785b18d83690>
    >>> transport.open()
    Could not connect to any of [('::1', 9090, 0, 0), ('127.0.0.1', 9090)]
    Traceback (most recent call last):
      File "<stdin>", line 1, in <module>
      File "/home/fishy/work/test/venv/lib/python3.11/site-packages/thrift/transport/TSocket.py", line 149, in open
        raise TTransportException(type=TTransportException.NOT_OPEN, message=msg)
    thrift.transport.TTransport.TTransportException: Could not connect to any of [('::1', 9090, 0, 0), ('127.0.0.1', 9090)]
    >>> from thrift.protocol import fastbinary
    >>> fastbinary
    <module 'thrift.protocol.fastbinary' from '/home/fishy/work/test/venv/lib/python3.11/site-packages/thrift/protocol/fastbinary.cpython-311-x86_64-linux-gnu.so'>
    >>> fastbinary.decode_compact
    <built-in function decode_compact>
    >>>

If we want to merge this version, I'll enable pending publishing with
`pypi.yml` from this repo on pypi [1].

[1]: https://pypi.org/manage/account/publishing/
2024-02-19 09:26:06 -08:00
Yuxuan 'fishy' Wang
875178c0ab THRIFT-5744: Switch to slog for go library
Client: go
2024-02-06 15:46:57 -08:00
Yuxuan 'fishy' Wang
3dcc0cdeb9 Revert "Bump upload/download-artifacts from v3 to v4"
This reverts commit 8540066800.

https://github.com/actions/upload-artifact/issues/478 will impact us.
2024-01-09 12:58:47 -08:00
Yuxuan 'fishy' Wang
8540066800 Bump upload/download-artifacts from v3 to v4
This is the combination of https://github.com/apache/thrift/pull/2910 &
https://github.com/apache/thrift/pull/2912. It looks like these 2
changes need to be done together, doing them individually will break CI.
2024-01-03 10:24:06 -08:00
dependabot[bot]
30c2845fdc Bump actions/setup-python from 4 to 5
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](https://github.com/actions/setup-python/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-01-03 09:31:55 -08:00
dependabot[bot]
1ebfcf8e0b Bump actions/setup-go from 4 to 5
Bumps [actions/setup-go](https://github.com/actions/setup-go) from 4 to 5.
- [Release notes](https://github.com/actions/setup-go/releases)
- [Commits](https://github.com/actions/setup-go/compare/v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-go
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2024-01-03 09:20:40 -08:00
Yuxuan 'fishy' Wang
022d0278eb Github Actions: Remove --disable-tests
With --disable-tests, for example for Go the `make check` under `lib/go`
would only run unit tests under `lib/go/thrift` but not the unit tests
under `lib/go/test`.

Also some changes in lib/go/test/fuzz/Makefile.am so it works in both go
1.20 and 1.21 (The current state breaks in 1.21 but because of
`--disable-tests` we never noticed that).
2023-12-04 06:57:25 -08:00
dependabot[bot]
f4b6d96008 Bump actions/setup-java from 3 to 4
Bumps [actions/setup-java](https://github.com/actions/setup-java) from 3 to 4.
- [Release notes](https://github.com/actions/setup-java/releases)
- [Commits](https://github.com/actions/setup-java/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/setup-java
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
2023-12-03 22:00:40 +01:00
Jiayu Liu
fbfa52cd04 THRIFT-5741: use rust 1.65 (#2870)
* use rust 1.65

* fix clippy

* fix alert

* fix protocol

* fix one more dereference

* fix more lint

* fix over-fix

* fix match &*server_type {
2023-11-07 13:47:24 +08:00
Jiayu Liu
d40dd7203e use gradle 8.4 (#2869) 2023-10-19 08:37:49 +08:00