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

Setup connection to Xilinx license server. #1918

Merged
merged 19 commits into from
Oct 25, 2022
Merged

Setup connection to Xilinx license server. #1918

merged 19 commits into from
Oct 25, 2022

Conversation

mithro
Copy link
Contributor

@mithro mithro commented Apr 25, 2022

Signed-off-by: Tim 'mithro' Ansell me@mith.ro

@mithro mithro linked an issue Apr 26, 2022 that may be closed by this pull request
@mithro
Copy link
Contributor Author

mithro commented Apr 28, 2022

@kgugala - Any update here?

@mithro
Copy link
Contributor Author

mithro commented May 4, 2022

@kgugala - Ping?

@pgielda
Copy link

pgielda commented May 5, 2022

This will be solved in a slightly different way, we are adding a new functionality to the runners to alloe tunneling, there will be a separate PR for that.
Backgrouns jobs will also be fixed, @kamilrakoczy has an idea on how to solve that.

@mithro
Copy link
Contributor Author

mithro commented May 24, 2022

@kgugala - What is the state here?

@kgugala
Copy link
Contributor

kgugala commented Jun 6, 2022

the functionality is in place, we can access the license server from a CI (including PRs). I've restarted CI here (it was red due to a random failure)

@kgugala kgugala marked this pull request as ready for review June 6, 2022 22:40
@kgugala
Copy link
Contributor

kgugala commented Jun 6, 2022

Oh, I have to sign-off commits

@kgugala
Copy link
Contributor

kgugala commented Jun 6, 2022

I've rebased the PR. Once the CI is green we can land it

@hansfbaier
Copy link
Collaborator

Great 👍🏻 can't wait to see K480T running. I already got Linux with DDR3 running on it

@hansfbaier
Copy link
Collaborator

@kgugala The error in this CI for artix7 is potentially spurious (some sort of timeout):

2022-06-07T02:17:13.2996817Z �[32;1m02:17:13�[0m | 2022-06-07T02:17:13 - xc7a200tffg1156-1/065b-gtp-common-pips      - 2h15m: make[3]: Leaving directory '/root/prjxray/prjxray/fuzzers/065b-gtp-common-pips'
2022-06-07T02:17:13.2997523Z �[32;1m02:17:13�[0m | 2022-06-07T02:17:13 - xc7a200tffg1156-1/065b-gtp-common-pips      - 2h15m: ERROR: timeout (max 10, got 11)
2022-06-07T02:17:13.2998136Z �[32;1m02:17:13�[0m | 2022-06-07T02:17:13 - xc7a200tffg1156-1/065b-gtp-common-pips      - 2h15m: ERROR: timeout

Also the spartan7 looks like a problem in the CI environment to me:

2022-06-07T00:45:54.0325065Z �[32;1m00:45:54�[0m | 2022-06-07T00:45:54 - xc7s50fgga484-1/050-pip-seed              - 1h43m: Still running (1m:183.50%, 5m:176.20%, 15m:175.98% Mem:42.0Gi used, 581.0Gi free).
2022-06-07T00:45:54.0326117Z �[32;1m00:45:54�[0m | 2022-06-07T00:45:54 - xc7s50fgga484-1/050-pip-seed              - 1h43m: make---sh---bash---make-+-21*[sh---bash-+-grep]
2022-06-07T00:45:54.0326787Z �[32;1m00:45:54�[0m | 2022-06-07T00:45:54 - xc7s50fgga484-1/050-pip-seed              - 1h43m:                         |               |-tee]
2022-06-07T00:45:54.0327633Z �[32;1m00:45:54�[0m | 2022-06-07T00:45:54 - xc7s50fgga484-1/050-pip-seed              - 1h43m:                         |               `-vivado.sh---vivado---loader---vivado]
2022-06-07T00:45:54.0328290Z �[32;1m00:45:54�[0m | 2022-06-07T00:45:54 - xc7s50fgga484-1/050-pip-seed              - 1h43m:                         |-5*[sh---bash-+-grep]
2022-06-07T00:45:54.0328900Z �[32;1m00:45:54�[0m | 2022-06-07T00:45:54 - xc7s50fgga484-1/050-pip-seed              - 1h43m:                         |              |-tee]
2022-06-07T00:45:54.0329683Z �[32;1m00:45:54�[0m | 2022-06-07T00:45:54 - xc7s50fgga484-1/050-pip-seed              - 1h43m:                         |              `-vivado.sh---vivado---loader---vivado---vivado---loader---parallel_synth_]
2022-06-07T00:45:54.0330447Z �[32;1m00:45:54�[0m | 2022-06-07T00:45:54 - xc7s50fgga484-1/050-pip-seed              - 1h43m:                         `-3*[sh---bash---python3]
2022-06-07T00:45:54.2767300Z ##[error]Process completed with exit code 255.
2022-06-07T00:45:54.2767300Z ##[error]Process completed with exit code 255.
2022-06-07T00:45:54.2780496Z ##[error]This error indicates issues with communication with the worker instance.
2022-06-07T00:45:58.7485677Z ##[error]Ping was successful.
2022-06-07T00:45:59.1371248Z ##[error]Traceback (most recent call last):
2022-06-07T00:45:59.1372658Z ##[error]  File "vm_command.py", line 612, in <module>
2022-06-07T00:45:59.1373555Z ##[error]    main()
2022-06-07T00:45:59.1374667Z ##[error]  File "/usr/lib/python3/dist-packages/click/core.py", line 764, in __call__
2022-06-07T00:45:59.1375901Z ##[error]    return self.main(*args, **kwargs)
2022-06-07T00:45:59.1377475Z ##[error]  File "/usr/lib/python3/dist-packages/click/core.py", line 717, in main
2022-06-07T00:45:59.1378728Z ##[error]    rv = self.invoke(ctx)
2022-06-07T00:45:59.1380066Z ##[error]  File "/usr/lib/python3/dist-packages/click/core.py", line 956, in invoke
2022-06-07T00:45:59.1381462Z ##[error]    return ctx.invoke(self.callback, **ctx.params)
2022-06-07T00:45:59.1383033Z ##[error]  File "/usr/lib/python3/dist-packages/click/core.py", line 555, in invoke
2022-06-07T00:45:59.1384484Z ##[error]    return callback(*args, **kwargs)
2022-06-07T00:45:59.1385422Z ##[error]  File "vm_command.py", line 600, in main
2022-06-07T00:45:59.1386204Z ##[error]    detect_preempted_signal(instance_number)
2022-06-07T00:45:59.1386998Z ##[error]  File "vm_command.py", line 513, in detect_preempted_signal
2022-06-07T00:45:59.1387752Z ##[error]    operations_dict = json.loads(r)
2022-06-07T00:45:59.1388534Z ##[error]  File "/usr/lib/python3.7/json/__init__.py", line 341, in loads
2022-06-07T00:45:59.1389375Z ##[error]    raise TypeError(f'the JSON object must be str, bytes or bytearray, '
2022-06-07T00:45:59.1390219Z ##[error]TypeError: the JSON object must be str, bytes or bytearray, not Response

@hansfbaier
Copy link
Collaborator

@mithro Can this be merged? It looks like the kintex branch is green and the errors in the other branches have nothing to do with the license server integration.

@mithro
Copy link
Contributor Author

mithro commented Jul 6, 2022

@kgugala - Can you look at merging this?

Copy link
Contributor

@kgugala kgugala left a comment

Choose a reason for hiding this comment

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

Some minor comments, otherwise LGTM. Once you update the PR we're good to go

.github/workflows/scripts/xilinx.sh Outdated Show resolved Hide resolved
.github/workflows/scripts/xilinx.sh Outdated Show resolved Hide resolved
.github/workflows/scripts/xilinx.sh Outdated Show resolved Hide resolved
.github/workflows/scripts/xilinx.sh Show resolved Hide resolved
mithro and others added 3 commits August 19, 2022 02:19
Signed-off-by: Tim 'mithro' Ansell <me@mith.ro>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
Signed-off-by: Karol Gugala <kgugala@antmicro.com>
@kgugala kgugala merged commit 18d46ba into master Oct 25, 2022
@kgugala kgugala deleted the xilinx branch October 25, 2022 04:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New GitHub actions based CI does not provide Vivado license
4 participants