Skip to content

Extend CI tests to include 2.10 and master + add EE #339

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.

Sign up for GitHub

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

Jump to bottom
Open
Gerold103 opened this issue Jun 7, 2022 · 5 comments
Open

Extend CI tests to include 2.10 and master + add EE #339

Gerold103 opened this issue Jun 7, 2022 · 5 comments
Assignees
@Mons
Labels
qa

Comments

@Gerold103
Copy link
Collaborator

Gerold103 commented Jun 7, 2022 •
edited
Loading

I have no idea how to do it, so here is the ticket.

Update: only EE support is left. Preferably of multiple versions of EE.
@Gerold103 Gerold103 added the qa label Jun 7, 2022
@Totktonada
Copy link
Member

Regarding EE: we investigated this topic in golang connector and you can consider results in tarantool/go-tarantool#183.

Regarding 2.10: the best option is to add support for the new tarantool versions into setup-tarantool action. I'm working on it. Precisely, I'm working on ability to set exact versions (like 1.10.8) and then plan to work on 2.10 support. However you can consider tricks implemented in CI for the smtp module: it directly installs necessary tarantool versions.

@Totktonada
Copy link
Member

Update: I resolved tarantool/setup-tarantool#19, so '2.10' can be added into the testing matrix without extra workarounds.

Beware: Don't forget quotes around '2.10'. Otherwise it will be interpreted by the YAML parser as the number 2.1.

Gerold103 added a commit that referenced this issue Jun 27, 2022
@Gerold103
ci: include 2.10
20bf419 
Part of #339
@Gerold103 Gerold103 mentioned this issue Jun 27, 2022
Merged
Gerold103 added a commit that referenced this issue Jun 27, 2022
@Gerold103
ci: include 2.10
969005a 
Part of #339
@sergos sergos added the 1sp label Jul 12, 2022
@sergos
Copy link
Contributor

sergos commented Jul 12, 2022

Duplicate of #331?

@Gerold103
Copy link
Collaborator Author

Not exactly, I also want to include tarantool master here.

@Totktonada
Copy link
Member

Not exactly, I also want to include tarantool master here.

You can copy-paste tarantool/expirationd#117 for example.

Serpentian added a commit to Serpentian/vshard that referenced this issue Jul 18, 2022
@Serpentian
ci: add a debug tarantool version
8db98f8 
This patch inroduce CI testing on the latest git version of
Tarantool, which is built as `Debug`.

By default master branch is used. However, any other one can
be used too: just change everything after `debug-` to another
name.

Part of tarantool#339
@Serpentian Serpentian mentioned this issue Jul 18, 2022
Merged
Gerold103 pushed a commit that referenced this issue Jul 18, 2022
@Serpentian @Gerold103
ci: add a debug tarantool version
d7dab3e 
This patch inroduce CI testing on the latest git version of
Tarantool, which is built as `Debug`.

By default master branch is used. However, any other one can
be used too: just change everything after `debug-` to another
name.

Part of #339
@TarantoolBot TarantoolBot removed the 1sp label Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Mons Mons

Labels
qa
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@Mons @Totktonada @Gerold103 @sergos @TarantoolBot