nipyapi icon indicating copy to clipboard operation
nipyapi copied to clipboard

`deploy_flow_version` with `version=None` does not deploy the latest version

Open knguyen1 opened this issue 1 year ago • 2 comments

  • Nipyapi version: 0.19.1
  • NiFi version: 1.20.0
  • NiFi-Registry version: 1.20.0
  • Python version: 3.8
  • Operating System: Linux

Description

Deploy a flow's latest version to a nifi instance that doesn't have the PG (first deploy).

Line 750 of versioning.py:

target_flow = target_flow[0].versioned_flow_snapshot_metadata

assumes that the returned flows are sorted descending with the latest version at index 0. This is not the case.

A better implementation would be (line 738):

target_flow = sorted(flow_versions.versioned_flow_snapshot_metadata_set, key=lambda x: x.versioned_flow_snapshot_metadata.version, reverse=True)

What I Did

nipyapi.versioning.deploy_flow_version(
        process_group_id, (0, 0), bucket_id, flow_id, registry_id
    )

Urgency

Not urgent

knguyen1 avatar May 08 '23 18:05 knguyen1

Thanks for this report and suggested patch, I will include it in the next release

Chaffelson avatar Jul 14 '23 07:07 Chaffelson

#262 likely duplicate

ChrisSamo632 avatar Jul 18 '23 20:07 ChrisSamo632