feat: release pipeline #97
Conversation
|
we need to retrigger commit pipeline and pr pipeline after this |
| @@ -0,0 +1,21 @@ | |||
| #!/bin/bash -l | |||
There was a problem hiding this comment.
Shouldn't concourse/scripts/test.sh be deleted?
| # build wheel | ||
| pip3 wheel . -w ../greenplumpython_artifacts | ||
| popd | ||
| tar -czf greenplumpython_artifacts/greenplumpython.tar.gz greenplumpython_artifacts/*.whl |
There was a problem hiding this comment.
So user needs to install psycopg2 manually?
There was a problem hiding this comment.
what will be the full name of the wheel file on different platforms? On my arch, it is:
greenplum_python-1.0.0b2-py3-none-any.whl
There was a problem hiding this comment.
can we do a simple test here? Just install the wheel and do a import test.
| /home/gpadmin/greenplumpython_src/concourse/scripts/test.sh" | ||
| ;; | ||
| /home/gpadmin/greenplumpython_src/concourse/scripts/test_and_build.sh" | ||
| ;; |
There was a problem hiding this comment.
spaces at the end should be removed.
There was a problem hiding this comment.
spaces at the end should be removed.
Just as thorough as back in the days. 😉
There was a problem hiding this comment.
Yes, I am a heartless tail-space-finding machine.
| @@ -56,6 +56,27 @@ on_error: | |||
| alert_type: errored | |||
| #@ end | |||
|
|
|||
| #! Release trigger. For dev pipelines. No webhook | |||
There was a problem hiding this comment.
For dev pipelines. No webhook
Is the comment correct?
| unpack: true | ||
| - put: #@ conf["release_bin"] | ||
| params: | ||
| file: #@ conf["res_intermediates_bin"] + "/greenplumpython_*_*.tar.gz" |
There was a problem hiding this comment.
this won't work since the unpacked file will be a .whl file
You could change the release bucket and test it.
| source: | ||
| bucket: pivotal-gpdb-concourse-resources-prod | ||
| json_key: ((concourse-gcs-resources-service-account-key)) | ||
| regexp: greenplumpython/released/greenplumpython-(.*).tar.gz |
| #! Release trigger. For dev pipelines. No webhook | ||
| #@ def release_trigger(res_map): | ||
| #@ add_res_by_name(res_map, "greenplumpython_commit") | ||
| #@ add_res_by_name(res_map, "greenplumpython_release") |
There was a problem hiding this comment.
Are line 61 & 62 needed?
No description provided.