search

Hyperfoil / qDup

Lab automation and queuing scripting
Apache License 2.0
12 stars 12 forks source link

[Mac-os]: Qdup latest version fails to execute and old version throws intermittent errors #205

Closed ShivangGoswami closed 5 months ago

ShivangGoswami commented 12 months ago

Java Environment:

shivanggoswami@MacBook-Pro resource-sizing-qdup-scripts % java --version
openjdk 22-ea 2024-03-19
OpenJDK Runtime Environment (build 22-ea+19-1460)
OpenJDK 64-Bit Server VM (build 22-ea+19-1460, mixed mode, sharing

Qdup Version(cloned from maven repos):

Qdup Script: 

scripts:
    tryme:
    - sh: whoami
    - sh: date
    - sh: echo ${{message}}
hosts:
    server: root@localhost:1234
roles:
    test:
        hosts:
        - server
        run-scripts:
        - tryme
states:
    message: Hello, qDup!

Ssh server is a podman container:

shivanggoswami@MacBook-Pro ~ % podman ps -a
CONTAINER ID  IMAGE                       COMMAND     CREATED         STATUS         PORTS                 NAMES
1bdc29330340  localhost/sshserver:latest              11 minutes ago  Up 11 minutes  0.0.0.0:1234->22/tcp  ssh-server01

Scenarios:

block causing error:

  create-knative-serving-load:
      - for-each:
          name: index
          input: ${{=[...Array(${{count:1}}).keys()]}}
        then:
        - sh: echo ${{index}}
johnaohara commented 12 months ago

Thanks for sharing, looks like graalvm library on MacOS is calling a native method that no longer exists in JDK22.

I am not sure of Graal Truffle is supported on JDK22, or even if we expect qDup to work on JDK22. Could you retry with JDK21 or JDK17?

@willr3 what versions of the JDK have we tested qDup with?

ShivangGoswami commented 12 months ago

Java Version:

shivanggoswami@Shivangs-MacBook-Pro ~ % java -version
openjdk version "21" 2023-09-19
OpenJDK Runtime Environment (build 21+35-2513)
OpenJDK 64-Bit Server VM (build 21+35-2513, mixed mode, sharing)

Scenarios:

furthermore, for-each block is also working fine with latest version as well.

ShivangGoswami commented 12 months ago

Maybe we should define a support matrix for qDup in the readme file with the versions that were tested as of now, so that new adapters of qDup have a clear picture of qDup prerequisites before trying in different operating systems and architectures cc: @johnaohara @willr3 @stalep

johnaohara commented 12 months ago

Maybe we should define a support matrix for qDup in the readme file with the versions that were tested as of now, so that new adapters of qDup have a clear picture of qDup prerequisites before trying in different operating systems and architectures cc: @johnaohara @willr3 @stalep

I think this is a great idea to document what JDK versions qDup is know to work with