ood_core 0.16.0 → 0.17.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 392bbf30609185792c59bc9b83bad0d2d6861885db4264a4da39356cb64624b6
-  data.tar.gz: 76df75234a646147a8fed14367d6467ffabe289bfa8264ff02614b60f6463652
+  metadata.gz: dca336fb15a08ad68f556b8d33fb76887f5c0370a0eef63685a5770fbf073110
+  data.tar.gz: 410b08fee5e739b7444ca3054483a2758d43062af964168b3f32318489d19fa0
 SHA512:
-  metadata.gz: fd127307d7048a220bdd5bed28302e90708bb0a32c9c6905136ddb3664b8f0c5035510c528f255161c4d40c98a772d6200fa60c938aff10ea4a157fdf56d8d1f
-  data.tar.gz: f3c1ea0622da5b4387ee123843da2e0e955afd40d28e0f25e425fea010865896cdebf87171e2264ffbffbd743be8d1533971a25566251dcb8da5748191dcbeea
+  metadata.gz: e82540895495b9f09c92f413f8f39a894fb700122da195cd4224d68eb5eae30845f8692c6d440462bac2c4f45b0a3270e7bf5219ba4adecfc63baa3884b53d28
+  data.tar.gz: 39a441ede8e9b91e169b1aff0c1345a56c98aac1f25a1b07d873b20e66833ed46a534b5ef23f392f5d0213b7e5c80e942f62b0031f4fd2065c327116975fdf8b

data/.github/dependabot.yml

@@ -0,0 +1,8 @@
+version: 2
+updates:
+- package-ecosystem: bundler
+  directory: "/"
+  schedule:
+    interval: daily
+    time: "03:30"
+  open-pull-requests-limit: 10

data/CHANGELOG.md

@@ -6,6 +6,66 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+
+## [0.17.2] - 7-14-2021
+
+### Fixed
+
+- Fixed k8s adapter to only show Running pods as running in [300](https://github.com/OSC/ood_core/pull/300).
+
+## [0.17.1] - 6-14-2021
+
+### Fixed
+
+- Fixed [278](https://github.com/OSC/ood_core/pull/278) where unschedulable pods will now show up as
+  queued_held status.
+
+### Changed
+
+- KUBECONFIG now defaults to /dev/null in the kubernetes adapter in [292](https://github.com/OSC/ood_core/pull/292).
+
+### Added
+
+- Sites can now set `batch_connect.ssh_allow` on the cluster to disable the buttons to start
+  a shell session to compute nodes in [289](https://github.com/OSC/ood_core/pull/289).
+- `POD_PORT` is now available to jobs in the kubernetes adapter in [290](https://github.com/OSC/ood_core/pull/290).
+- Kubernetes pods now support a startProbe in [291](https://github.com/OSC/ood_core/pull/291).
+
+## [0.17.0] - 5-26-2021
+
+### Fixed
+
+- All Kubernetes resources now have the same labels in [280](https://github.com/OSC/ood_core/pull/280).
+- Kubernetes does not crash when no configmap is defined in [282](https://github.com/OSC/ood_core/pull/282).
+- Kubernetes will not specify init containers if there are none in
+  [284](https://github.com/OSC/ood_core/pull/284).
+
+### Added
+
+- Kubernetes, Slurm and Torque now support the script option `gpus_per_node` in
+  [266](https://github.com/OSC/ood_core/pull/266).
+- Kubernetes will now save the pod.yml into the staged root in
+  [277](https://github.com/OSC/ood_core/pull/277).
+- Kubernetes now allows for node selector in [264](https://github.com/OSC/ood_core/pull/264).
+- Kubernetes pods now have access the environment variable POD_NAMESPACE in
+  [275](https://github.com/OSC/ood_core/pull/275).
+- Kubernetes pods can now specify the image pull policy in [272](https://github.com/OSC/ood_core/pull/272).
+- Cluster config's batch_connect now support `ssh_allow` to disable sshing to compute
+  nodes per cluster in [286](https://github.com/OSC/ood_core/pull/286).
+- Kubernetes will now add the templated script content to a configmap in
+  [273](https://github.com/OSC/ood_core/pull/273).
+
+### Changed
+
+- Kubernetes username prefix no longer appends a - in [271](https://github.com/OSC/ood_core/pull/271).
+
+
+
+## [0.16.1] - 2021-04-23
+### Fixed
+- memorized some allow? variables to have better support around ACLS in
+  [267](https://github.com/OSC/ood_core/pull/267)
+
 ## [0.16.0] - 2021-04-20
 ### Fixed
 - tmux 2.7+ bug in the linux host adapter in [2.5.8](https://github.com/OSC/ood_core/pull/258)
@@ -300,8 +360,12 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 ### Added
 - Initial release!
 
-[Unreleased]: https://github.com/OSC/ood_core/compare/v0.16.0...HEAD
-[0.15.2]: https://github.com/OSC/ood_core/compare/v0.15.1...v0.16.0
+[Unreleased]: https://github.com/OSC/ood_core/compare/v0.17.2...HEAD
+[0.17.2]: https://github.com/OSC/ood_core/compare/v0.17.1...v0.17.2
+[0.17.1]: https://github.com/OSC/ood_core/compare/v0.17.0...v0.17.1
+[0.17.0]: https://github.com/OSC/ood_core/compare/v0.16.1...v0.17.0
+[0.16.1]: https://github.com/OSC/ood_core/compare/v0.16.0...v0.16.1
+[0.16.0]: https://github.com/OSC/ood_core/compare/v0.15.1...v0.16.0
 [0.15.1]: https://github.com/OSC/ood_core/compare/v0.15.0...v0.15.1
 [0.15.0]: https://github.com/OSC/ood_core/compare/v0.14.0...v0.15.0
 [0.14.0]: https://github.com/OSC/ood_core/compare/v0.13.0...v0.14.0

data/lib/ood_core/cluster.rb

@@ -78,7 +78,9 @@ module OodCore
     # Whether the login feature is allowed
     # @return [Boolean] is login allowed
     def login_allow?
-      allow? && !login_config.empty?
+      return @login_allow if defined?(@login_allow)
+
+      @login_allow = (allow? && !login_config.empty?)
     end
 
     # Build a job adapter from the job configuration
@@ -90,9 +92,11 @@ module OodCore
     # Whether the job feature is allowed based on the ACLs
     # @return [Boolean] is the job feature allowed
     def job_allow?
-      allow? &&
-        !job_config.empty? &&
-        build_acls(job_config.fetch(:acls, []).map(&:to_h)).all?(&:allow?)
+      return @job_allow if defined?(@job_allow)
+
+      @job_allow = (allow? && ! job_config.empty? && build_acls(
+        job_config.fetch(:acls, []).map(&:to_h)
+      ).all?(&:allow?))
     end
 
     # The batch connect template configuration used for this cluster
@@ -138,7 +142,18 @@ module OodCore
     # Whether this cluster is allowed to be used
     # @return [Boolean] whether cluster is allowed
     def allow?
-      acls.all?(&:allow?)
+      return @allow if defined?(@allow)
+
+      @allow = acls.all?(&:allow?)
+    end
+
+    # Whether this cluster supports SSH to batch connect nodes
+    # @return [Boolean, nil] whether cluster supports SSH to batch connect node
+    def batch_connect_ssh_allow?
+      return @batch_connect_ssh_allow if defined?(@batch_connect_ssh_allow)
+      return @batch_connect_ssh_allow = nil if batch_connect_config.nil?
+
+      @batch_connect_ssh_allow = batch_connect_config.fetch(:ssh_allow, nil)
     end
 
     # The comparison operator

data/lib/ood_core/job/adapters/kubernetes/batch.rb

@@ -23,7 +23,7 @@ class OodCore::Job::Adapters::Kubernetes::Batch
     @cluster = options.fetch(:cluster, 'open-ondemand')
     @mounts = options.fetch(:mounts, []).map { |m| m.to_h.symbolize_keys }
     @all_namespaces = options.fetch(:all_namespaces, false)
-    @username_prefix = options.fetch(:username_prefix, nil)
+    @username_prefix = options.fetch(:username_prefix, '')
     @namespace_prefix = options.fetch(:namespace_prefix, '')
 
     @using_context = false
@@ -45,6 +45,9 @@ class OodCore::Job::Adapters::Kubernetes::Batch
     raise ArgumentError, 'Must specify the script' if script.nil?
 
     resource_yml, id = generate_id_yml(script)
+    if !script.workdir.nil? && Dir.exist?(script.workdir)
+      File.open(File.join(script.workdir, 'pod.yml'), 'w') { |f| f.write resource_yml }
+    end
     call("#{formatted_ns_cmd} create -f -", stdin: resource_yml)
 
     id
@@ -146,7 +149,7 @@ class OodCore::Job::Adapters::Kubernetes::Batch
   end
 
   def k8s_username
-    username_prefix.nil? ? username : "#{username_prefix}-#{username}"
+    "#{username_prefix}#{username}"
   end
 
   def user
@@ -180,6 +183,7 @@ class OodCore::Job::Adapters::Kubernetes::Batch
       HOME: home_dir,
       GROUP: group,
       GID: run_as_group,
+      KUBECONFIG: '/dev/null',
     }
   end
 
@@ -189,10 +193,12 @@ class OodCore::Job::Adapters::Kubernetes::Batch
     native_data = script.native
     container = helper.container_from_native(native_data[:container], default_env)
     id = generate_id(container.name)
-    configmap = helper.configmap_from_native(native_data, id)
+    configmap = helper.configmap_from_native(native_data, id, script.content)
     init_containers = helper.init_ctrs_from_native(native_data[:init_containers], container.env)
     spec = OodCore::Job::Adapters::Kubernetes::Resources::PodSpec.new(container, init_containers: init_containers)
     all_mounts = native_data[:mounts].nil? ? mounts : mounts + native_data[:mounts]
+    node_selector = native_data[:node_selector].nil? ? {} : native_data[:node_selector]
+    gpu_type = native_data[:gpu_type].nil? ? "nvidia.com/gpu" : native_data[:gpu_type]
 
     template = ERB.new(File.read(resource_file), nil, '-')
 

data/lib/ood_core/job/adapters/kubernetes/helper.rb

@@ -53,7 +53,9 @@ class OodCore::Job::Adapters::Kubernetes::Helper
       cpu: container[:cpu],
       working_dir: container[:working_dir],
       restart_policy: container[:restart_policy],
-      image_pull_secret: container[:image_pull_secret]
+      image_pull_policy: container[:image_pull_policy],
+      image_pull_secret: container[:image_pull_secret],
+      startup_probe: container[:startup_probe],
     )
   end
 
@@ -80,10 +82,18 @@ class OodCore::Job::Adapters::Kubernetes::Helper
   #   the input configmap hash
   # @param id [#to_s]
   #   the id to use for giving the configmap a name
+  # @param script_content [#to_s]
+  #   the batch script content
   # @return  [OodCore::Job::Adapters::Kubernetes::Resources::ConfigMap]
-  def configmap_from_native(native, id)
-    configmap = native.fetch(:configmap, nil)
-    return nil if configmap.nil?
+  def configmap_from_native(native, id, script_content)
+    configmap = native.fetch(:configmap, {})
+    configmap[:files] ||= []
+    configmap[:files] << {
+      filename: 'script.sh',
+      data: script_content,
+      mount_path: '/ood/script.sh',
+      sub_path: 'script.sh',
+    } unless configmap[:files].any? { |f| f[:filename] == 'script.sh' }
 
     OodCore::Job::Adapters::Kubernetes::Resources::ConfigMap.new(
       configmap_name(id),
@@ -140,7 +150,7 @@ class OodCore::Job::Adapters::Kubernetes::Helper
     {
       id: json_data.dig(:metadata, :name).to_s,
       job_name: name_from_metadata(json_data.dig(:metadata)),
-      status: pod_status_from_json(json_data),
+      status: OodCore::Job::Status.new(state: pod_status_from_json(json_data)),
       job_owner: job_owner_from_json(json_data, ns_prefix),
       submission_time: submission_time(json_data),
       dispatch_time: dispatch_time(json_data),
@@ -230,15 +240,21 @@ class OodCore::Job::Adapters::Kubernetes::Helper
   def submission_time(json_data)
     status = json_data.dig(:status)
     start = status.dig(:startTime)
+    creation = json_data.dig(:metadata, :creationTimestamp)
 
-    if start.nil?
+    if !creation.nil?
+      str = creation
+    elsif !start.nil?
+      str = start
+    else
       # the pod is in some pending state limbo
       conditions = status.dig(:conditions)
+      return nil if conditions.nil?
+      return nil if conditions.size == 0
       # best guess to start time is just the first condition's
       # transition time
       str = conditions[0].dig(:lastTransitionTime)
-    else
-      str = start
+      return nil if str.nil?
     end
 
     DateTime.parse(str).to_time.to_i
@@ -246,11 +262,19 @@ class OodCore::Job::Adapters::Kubernetes::Helper
 
   def pod_status_from_json(json_data)
     phase = json_data.dig(:status, :phase)
+    conditions = json_data.dig(:status, :conditions)
+    container_statuses = json_data.dig(:status, :containerStatuses)
+    unschedulable = conditions.to_a.any? { |c| c.dig(:reason) == "Unschedulable" }
+    ready = !container_statuses.to_a.empty? && container_statuses.to_a.all? { |s| s.dig(:ready) == true }
+    return "running" if ready
+
     state = case phase
-            when "Running"
-              "running"
             when "Pending"
-              "queued"
+              if unschedulable
+                "queued_held"
+              else
+                "queued"
+              end
             when "Failed"
               "suspended"
             when "Succeeded"
@@ -260,8 +284,6 @@ class OodCore::Job::Adapters::Kubernetes::Helper
             else
               "undetermined"
             end
-
-    OodCore::Job::Status.new(state: state)
   end
 
   def terminated_state(status)

data/lib/ood_core/job/adapters/kubernetes/resources.rb

@@ -33,13 +33,36 @@ module OodCore::Job::Adapters::Kubernetes::Resources
     end
   end
 
+  class TCPProbe
+    attr_accessor :port, :initial_delay_seconds, :failure_threshold, :period_seconds
+
+    def initialize(port, data)
+      data ||= {}
+      @port = port
+      @initial_delay_seconds = data[:initial_delay_seconds] || 2
+      @failure_threshold = data[:failure_threshold] || 5
+      @period_seconds = data[:period_seconds] || 5
+    end
+
+    def to_h
+      {
+        port: port,
+        initial_delay_seconds: initial_delay_seconds,
+        failure_threshold: failure_threshold,
+        period_seconds: period_seconds,
+      }
+    end
+  end
+
   class Container
     attr_accessor :name, :image, :command, :port, :env, :memory, :cpu, :working_dir,
-                  :restart_policy, :image_pull_secret, :supplemental_groups
+                  :restart_policy, :image_pull_policy, :image_pull_secret, :supplemental_groups,
+                  :startup_probe
 
     def initialize(
         name, image, command: [], port: nil, env: {}, memory: "4Gi", cpu: "1",
-        working_dir: "", restart_policy: "Never", image_pull_secret: nil, supplemental_groups: []
+        working_dir: "", restart_policy: "Never", image_pull_policy: nil, image_pull_secret: nil, supplemental_groups: [],
+        startup_probe: {}
       )
       raise ArgumentError, "containers need valid names and images" unless name && image
 
@@ -52,8 +75,10 @@ module OodCore::Job::Adapters::Kubernetes::Resources
       @cpu = cpu.nil? ? "1" : cpu
       @working_dir = working_dir.nil? ? "" : working_dir
       @restart_policy = restart_policy.nil? ? "Never" : restart_policy
+      @image_pull_policy = image_pull_policy.nil? ? "IfNotPresent" : image_pull_policy
       @image_pull_secret = image_pull_secret
       @supplemental_groups = supplemental_groups.nil? ? [] : supplemental_groups
+      @startup_probe = TCPProbe.new(@port, startup_probe)
     end
 
     def ==(other)
@@ -66,8 +91,10 @@ module OodCore::Job::Adapters::Kubernetes::Resources
         cpu == other.cpu &&
         working_dir == other.working_dir &&
         restart_policy == other.restart_policy &&
+        image_pull_policy == other.image_pull_policy &&
         image_pull_secret == other.image_pull_secret &&
-        supplemental_groups == other.supplemental_groups
+        supplemental_groups == other.supplemental_groups &&
+        startup_probe.to_h == other.startup_probe.to_h
     end
   end
 

data/lib/ood_core/job/adapters/kubernetes/templates/pod.yml.erb

@@ -39,7 +39,7 @@ spec:
   containers:
   - name: "<%= spec.container.name %>"
     image: <%= spec.container.image %>
-    imagePullPolicy: IfNotPresent
+    imagePullPolicy: <%= spec.container.image_pull_policy %>
     <%- unless spec.container.working_dir.empty? -%>
     workingDir: "<%= spec.container.working_dir %>"
     <%- end -%>
@@ -48,6 +48,14 @@ spec:
       valueFrom:
         fieldRef:
           fieldPath: metadata.name
+    - name: POD_NAMESPACE
+      valueFrom:
+        fieldRef:
+          fieldPath: metadata.namespace
+    <%- unless spec.container.port.nil? -%>
+    - name: POD_PORT
+      value: "<%= spec.container.port %>"
+    <%- end -%>
     <%- spec.container.env.each_pair do |name, value| -%>
     - name: <%= name %>
       value: "<%= value %>"
@@ -61,9 +69,16 @@ spec:
     <%- unless spec.container.port.nil? -%>
     ports:
     - containerPort: <%= spec.container.port %>
+    startupProbe:
+      tcpSocket:
+        port: <%= spec.container.startup_probe.port %>
+      initialDelaySeconds: <%= spec.container.startup_probe.initial_delay_seconds %>
+      failureThreshold: <%= spec.container.startup_probe.failure_threshold %>
+      periodSeconds: <%= spec.container.startup_probe.period_seconds %>
     <%- end -%>
-    <%- if configmap.mounts? || !all_mounts.empty? -%>
+    <%- if !all_mounts.empty? || (!configmap.nil? && configmap.mounts?) -%>
     volumeMounts:
+    <%- unless configmap.nil? -%>
     <%- configmap.files.each do |file| -%>
       <%- next if file.mount_path.nil? -%>
     - name: configmap-volume
@@ -72,6 +87,7 @@ spec:
       subPath: <%= file.sub_path %>
       <%- end # end unless file.sub_path.nil? -%>
     <%- end # end configmap.files.each -%>
+    <%- end # unless configmap.nil? -%>
     <%- all_mounts.each do |mount| -%>
     - name: <%= mount[:name] %>
       mountPath: <%= mount[:destination_path] %>
@@ -81,25 +97,36 @@ spec:
       limits:
         memory: "<%= spec.container.memory %>"
         cpu: "<%= spec.container.cpu %>"
+        <%- unless script.gpus_per_node.nil? -%>
+        <%= gpu_type %>: <%= script.gpus_per_node %>
+        <%- end -%>
       requests:
         memory: "<%= spec.container.memory %>"
         cpu: "<%= spec.container.cpu %>"
+        <%- unless script.gpus_per_node.nil? -%>
+        <%= gpu_type %>: <%= script.gpus_per_node %>
+        <%- end -%>
     securityContext:
       allowPrivilegeEscalation: false
       capabilities:
         drop:
         - all
       privileged: false
-  <%- unless spec.init_containers.nil? -%>
+  <%- unless spec.init_containers.empty? -%>
   initContainers:
   <%- spec.init_containers.each do |ctr| -%>
   - name: "<%= ctr.name %>"
     image: "<%= ctr.image %>"
+    imagePullPolicy: <%= ctr.image_pull_policy %>
     env:
     - name: POD_NAME
       valueFrom:
         fieldRef:
           fieldPath: metadata.name
+    - name: POD_NAMESPACE
+      valueFrom:
+        fieldRef:
+          fieldPath: metadata.namespace
     <%- ctr.env.each_pair do |name, value| -%>
     - name: <%= name %>
       value: "<%= value %>"
@@ -108,8 +135,9 @@ spec:
     <%- ctr.command.each do |cmd| -%>
     - "<%= cmd %>"
     <%- end # command loop -%>
-    <%- if configmap.init_mounts? || !all_mounts.empty? -%>
+    <%- if !all_mounts.empty? || (!configmap.nil? && configmap.init_mounts?) -%>
     volumeMounts:
+    <%- unless configmap.nil? -%>
     <%- configmap.files.each do |file| -%>
     <%- next if file.init_mount_path.nil? -%>
     - name: configmap-volume
@@ -118,6 +146,7 @@ spec:
       subPath: <%= file.init_sub_path %>
       <%- end # end unless file.sub_path.nil? -%>
     <%- end # end configmap.files.each -%>
+    <%- end # unless configmap.nil? -%>
     <%- all_mounts.each do |mount| -%>
     - name: <%= mount[:name] %>
       mountPath: <%= mount[:destination_path] %>
@@ -152,6 +181,12 @@ spec:
   <%- end # if mount is [host,nfs] -%>
   <%- end # for each mount -%>
   <%- end # (configmap.to_s.empty? || all_mounts.empty?) -%>
+  <%- unless node_selector.empty? -%>
+  nodeSelector:
+  <%- node_selector.each_pair do |key, value| -%>
+    <%= key %>: "<%= value %>"
+  <%- end # node_selector.each_pair -%>
+  <%- end #unless node_selector.empty? -%>
 ---
 <%- unless spec.container.port.nil? -%>
 apiVersion: v1
@@ -161,6 +196,8 @@ metadata:
   namespace: <%= namespace %>
   labels:
     job: <%= id %>
+    app.kubernetes.io/name: <%= container.name %>
+    app.kubernetes.io/managed-by: open-ondemand
 spec:
   selector:
     job: <%= id %>
@@ -170,8 +207,8 @@ spec:
     targetPort: <%= spec.container.port %>
   type: NodePort
 <%- end # end for service -%>
----
 <%- unless configmap.nil? -%>
+---
 apiVersion: v1
 kind: ConfigMap
 metadata:
@@ -179,6 +216,8 @@ metadata:
   namespace: <%= namespace %>
   labels:
     job: <%= id %>
+    app.kubernetes.io/name: <%= container.name %>
+    app.kubernetes.io/managed-by: open-ondemand
 data:
 <%- configmap.files.each do |file| -%>
   <%- next if file.data.nil? || file.filename.nil? -%>

data/lib/ood_core/job/adapters/slurm.rb

@@ -423,6 +423,7 @@ module OodCore
           args.concat ["-t", seconds_to_duration(script.wall_time)] unless script.wall_time.nil?
           args.concat ['-a', script.job_array_request] unless script.job_array_request.nil?
           args.concat ['--qos', script.qos] unless script.qos.nil?
+          args.concat ['--gpus-per-node', script.gpus_per_node] unless script.gpus_per_node.nil?
           # ignore nodes, don't know how to do this for slurm
 
           # Set dependencies

data/lib/ood_core/job/adapters/torque.rb

@@ -159,6 +159,8 @@ module OodCore
             args.concat ["-l", "walltime=#{seconds_to_duration(script.wall_time)}"] unless script.wall_time.nil?
             args.concat ['-t', script.job_array_request] unless script.job_array_request.nil?
             args.concat ['-l', "qos=#{script.qos}"] unless script.qos.nil?
+            args.concat ['-l', "gpus=#{script.gpus_per_node}"] unless script.gpus_per_node.nil?
+
             # Set environment variables
             env = script.job_environment.to_h
             args.concat ["-v", env.keys.join(",")] unless env.empty?

data/lib/ood_core/job/script.rb

@@ -103,6 +103,10 @@ module OodCore
       # @return [String, nil] qos
       attr_reader :qos
 
+      # The GPUs per node for the job
+      # @return [Integer, nil] gpus per node
+      attr_reader :gpus_per_node
+
       # Object detailing any native specifications that are implementation specific
       # @note Should not be used at all costs.
       # @return [Object, nil] native specifications
@@ -136,6 +140,7 @@ module OodCore
       # @param accounting_id [#to_s, nil] accounting id
       # @param job_array_request [#to_s, nil] job array request
       # @param qos [#to_s, nil] qos
+      # @param gpus_per_node [#to_i, nil] gpus per node
       # @param native [Object, nil] native specifications
       # @param copy_environment [Boolean, nil] copy the environment
       def initialize(content:, args: nil, submit_as_hold: nil, rerunnable: nil,
@@ -145,7 +150,7 @@ module OodCore
                      output_path: nil, error_path: nil, reservation_id: nil,
                      queue_name: nil, priority: nil, start_time: nil,
                      wall_time: nil, accounting_id: nil, job_array_request: nil,
-                     qos: nil, native: nil, copy_environment: nil, **_)
+                     qos: nil, gpus_per_node: nil, native: nil, copy_environment: nil, **_)
         @content = content.to_s
 
         @submit_as_hold      = submit_as_hold
@@ -170,6 +175,7 @@ module OodCore
         @accounting_id      = accounting_id     && accounting_id.to_s
         @job_array_request  = job_array_request && job_array_request.to_s
         @qos                = qos               && qos.to_s
+        @gpus_per_node      = gpus_per_node     && gpus_per_node.to_i
         @native             = native
         @copy_environment   = (copy_environment.nil?) ? nil : !! copy_environment
       end
@@ -200,6 +206,7 @@ module OodCore
           accounting_id:       accounting_id,
           job_array_request:   job_array_request,
           qos:                 qos,
+          gpus_per_node:       gpus_per_node,
           native:              native,
           copy_environment:    copy_environment
         }

data/lib/ood_core/version.rb

@@ -1,4 +1,4 @@
 module OodCore
   # The current version of {OodCore}
-  VERSION = "0.16.0"
+  VERSION = "0.17.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ood_core
 version: !ruby/object:Gem::Version
-  version: 0.16.0
+  version: 0.17.2
 platform: ruby
 authors:
 - Eric Franz
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-04-20 00:00:00.000000000 Z
+date: 2021-07-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ood_support
@@ -160,6 +160,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/dependabot.yml"
 - ".github/workflows/test.yml"
 - ".gitignore"
 - ".rspec"