From 4246877c4655c7c68816e6b0183cd08ff5f82ba8 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Tue, 9 Jul 2024 15:43:36 +0530 Subject: [PATCH 01/22] Added code to run builds against our existing LTI releases or branches --- .github/workflows/build.yaml | 13 +++++++++++++ .github/workflows/cronJob.yaml | 9 +++++++++ 2 files changed, 22 insertions(+) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 46e3bb7d9..78cc69d3f 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -17,6 +17,11 @@ on: required: true type: boolean default: false + refLTITag: + description: 'Reference LTI Tag' + type: string + required: true + default: main workflow_dispatch: inputs: useLocalPlugin: @@ -29,6 +34,11 @@ on: type: string required: true default: main + refLTITag: + description: 'Reference LTI Tag' + type: string + required: true + default: main push: branches: '**' pull_request: @@ -55,6 +65,7 @@ jobs: USE_LOCAL_PLUGIN: ${{ inputs.useLocalPlugin || false }} REF_LSP4IJ: ${{ inputs.refLsp4ij }} LSP4IJ_BRANCH: ${{ inputs.lsp4ijBranch || 'latest' }} + REF_LTI_TAG: ${{ inputs.refLTITag }} steps: - name: Configure pagefile if: contains(matrix.os, 'windows') @@ -66,7 +77,9 @@ jobs: - name: 'Checkout liberty-tools-intellij' uses: actions/checkout@v3 with: + repository: OpenLiberty/Liberty-tools-intellij path: liberty-tools-intellij + ref: ${{ env.REF_LTI_TAG }} - name: 'Install required integration test software' working-directory: ./liberty-tools-intellij run: bash ./src/test/resources/ci/scripts/setup.sh diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 659c8304c..6a4645c48 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -1,6 +1,7 @@ name: Cron Job on: + push: workflow_dispatch: schedule: # The job runs at 15:45 UTC every Monday through Friday, which is 9:15 PM IST and 11:45 AM EST. @@ -94,16 +95,24 @@ jobs: strategy: fail-fast: false matrix: + # Existing LTI release tags can be added to obtain build results. + tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' pr_details: ${{ fromJson(needs.fetch_all_pull_request_shas.outputs.pr_details) }} with: useLocalPlugin: true refLsp4ij: ${{ matrix.pr_details.sha }} lsp4ijBranch: PR-${{ matrix.pr_details.number }} + refLTITag: ${{ matrix.tag }} name: Running PR # Run the LTI Tests against lsp4ij main branch call-build-workflow-for-lsp4ij-main-branch: uses: ./.github/workflows/build.yaml + strategy: + fail-fast: false + matrix: + # Existing LTI release tags can be added to obtain build results. + tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' with: useLocalPlugin: true refLsp4ij: main From a700314e3867c086aa670eddb447c7e3d21809fb Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Tue, 9 Jul 2024 15:46:10 +0530 Subject: [PATCH 02/22] Added required 'refLTITag' --- .github/workflows/cronJob.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 6a4645c48..28837411f 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -117,6 +117,7 @@ jobs: useLocalPlugin: true refLsp4ij: main lsp4ijBranch: main + refLTITag: ${{ matrix.tag }} name: Run LTI tests for Lsp4ij Main branch # Send slack notification for build results From f901025a61dd27c04acfa0c9f804b5198b186783 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Tue, 9 Jul 2024 15:56:16 +0530 Subject: [PATCH 03/22] Removed repository --- .github/workflows/build.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 78cc69d3f..9deabfa88 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -77,7 +77,6 @@ jobs: - name: 'Checkout liberty-tools-intellij' uses: actions/checkout@v3 with: - repository: OpenLiberty/Liberty-tools-intellij path: liberty-tools-intellij ref: ${{ env.REF_LTI_TAG }} - name: 'Install required integration test software' From b1be2026a203f24f457d869f6837144d7f45c6c1 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Tue, 9 Jul 2024 16:07:17 +0530 Subject: [PATCH 04/22] Added 'main' for testing --- .github/workflows/cronJob.yaml | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 28837411f..c7f05048d 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -96,7 +96,8 @@ jobs: fail-fast: false matrix: # Existing LTI release tags can be added to obtain build results. - tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' + # tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' + tag: [ 'main' ] pr_details: ${{ fromJson(needs.fetch_all_pull_request_shas.outputs.pr_details) }} with: useLocalPlugin: true @@ -112,7 +113,8 @@ jobs: fail-fast: false matrix: # Existing LTI release tags can be added to obtain build results. - tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' + # tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' + tag: [ 'main' ] with: useLocalPlugin: true refLsp4ij: main From 78779710603723ede8c890ff3e47aa6b9b431b6d Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Tue, 9 Jul 2024 16:25:39 +0530 Subject: [PATCH 05/22] Specified repo path to get build success on both cron job and default build --- .github/workflows/cronJob.yaml | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index c7f05048d..28837411f 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -96,8 +96,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags can be added to obtain build results. - # tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' - tag: [ 'main' ] + tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' pr_details: ${{ fromJson(needs.fetch_all_pull_request_shas.outputs.pr_details) }} with: useLocalPlugin: true @@ -113,8 +112,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags can be added to obtain build results. - # tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' - tag: [ 'main' ] + tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' with: useLocalPlugin: true refLsp4ij: main From f32bf4b59bf50e6d8c5da4c614548503aaf1efa0 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Tue, 9 Jul 2024 17:02:08 +0530 Subject: [PATCH 06/22] Added comments and specified repo path --- .github/workflows/cronJob.yaml | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 28837411f..10fa2486a 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -103,7 +103,8 @@ jobs: refLsp4ij: ${{ matrix.pr_details.sha }} lsp4ijBranch: PR-${{ matrix.pr_details.number }} refLTITag: ${{ matrix.tag }} - name: Running PR + name: Run PR + # Run the LTI Tests against lsp4ij main branch call-build-workflow-for-lsp4ij-main-branch: @@ -112,13 +113,13 @@ jobs: fail-fast: false matrix: # Existing LTI release tags can be added to obtain build results. - tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' + tag: [ 'lsp4ij-market-0.0.2-integration' ] with: useLocalPlugin: true refLsp4ij: main lsp4ijBranch: main refLTITag: ${{ matrix.tag }} - name: Run LTI tests for Lsp4ij Main branch + name: Run Lsp4ij Main # Send slack notification for build results call-build-workflow-slack-notification: From 72cf004d52d2ab519ac2c5f77df09e1ede043b44 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Tue, 9 Jul 2024 17:13:56 +0530 Subject: [PATCH 07/22] Removed added 'push' for testing cron job --- .github/workflows/cronJob.yaml | 1 - 1 file changed, 1 deletion(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 10fa2486a..f1de1ecc6 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -1,7 +1,6 @@ name: Cron Job on: - push: workflow_dispatch: schedule: # The job runs at 15:45 UTC every Monday through Friday, which is 9:15 PM IST and 11:45 AM EST. From 43b640d54f154f15a28961fc3d4508a4b3550b7e Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Fri, 12 Jul 2024 19:03:44 +0530 Subject: [PATCH 08/22] Update cronJob.yaml --- .github/workflows/cronJob.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index f1de1ecc6..10fa2486a 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -1,6 +1,7 @@ name: Cron Job on: + push: workflow_dispatch: schedule: # The job runs at 15:45 UTC every Monday through Friday, which is 9:15 PM IST and 11:45 AM EST. From 3cc954ddfdab14c1fbeaedc1f5e5c63e2c1a1c68 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Fri, 12 Jul 2024 19:05:44 +0530 Subject: [PATCH 09/22] Update cronJob.yaml --- .github/workflows/cronJob.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 10fa2486a..6efddc993 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -113,7 +113,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags can be added to obtain build results. - tag: [ 'lsp4ij-market-0.0.2-integration' ] + tag: [] with: useLocalPlugin: true refLsp4ij: main From 2e4e5085fec56ecd3405ac6a6ba2414efcdbeee0 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Fri, 12 Jul 2024 19:06:31 +0530 Subject: [PATCH 10/22] Update cronJob.yaml --- .github/workflows/cronJob.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 6efddc993..5cf7b61d0 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -113,7 +113,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags can be added to obtain build results. - tag: [] + tag: [''] with: useLocalPlugin: true refLsp4ij: main From 87c22c4ff8dd187826e74a79352f4ee3b4d0e3c1 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Fri, 12 Jul 2024 19:24:44 +0530 Subject: [PATCH 11/22] Update cronJob.yaml --- .github/workflows/cronJob.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 5cf7b61d0..56cac0946 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -96,7 +96,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags can be added to obtain build results. - tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' + tag: [ '' ] # Can specify tags or branches such as '24.0.6' and 'main' pr_details: ${{ fromJson(needs.fetch_all_pull_request_shas.outputs.pr_details) }} with: useLocalPlugin: true From 197ea8d2aed3cbde1526c6bda6c49d0bf519890c Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Fri, 12 Jul 2024 20:09:03 +0530 Subject: [PATCH 12/22] Required given back to false for 'refLTITag' --- .github/workflows/build.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 9deabfa88..c610855ea 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -37,7 +37,7 @@ on: refLTITag: description: 'Reference LTI Tag' type: string - required: true + required: false default: main push: branches: '**' From 6bbe8f9b9d84acab2e4757a4b87a8b7ce8773004 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Fri, 12 Jul 2024 20:30:44 +0530 Subject: [PATCH 13/22] Addressed review comments --- .github/workflows/build.yaml | 4 ++-- .github/workflows/cronJob.yaml | 5 ++--- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index c610855ea..38ae3f426 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -18,7 +18,7 @@ on: type: boolean default: false refLTITag: - description: 'Reference LTI Tag' + description: 'Reference LTI Tag/Branch' type: string required: true default: main @@ -35,7 +35,7 @@ on: required: true default: main refLTITag: - description: 'Reference LTI Tag' + description: 'Reference LTI Tag/Branch' type: string required: false default: main diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 56cac0946..332b413c7 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -1,7 +1,6 @@ name: Cron Job on: - push: workflow_dispatch: schedule: # The job runs at 15:45 UTC every Monday through Friday, which is 9:15 PM IST and 11:45 AM EST. @@ -95,7 +94,7 @@ jobs: strategy: fail-fast: false matrix: - # Existing LTI release tags can be added to obtain build results. + # Existing LTI release tags and branches can be added to obtain build results.If the tag array is empty, it will default to 'main'. However, if there is at least one tag or branch in the tag array and you need to run on 'main' as well, make sure to add 'main' to the array. tag: [ '' ] # Can specify tags or branches such as '24.0.6' and 'main' pr_details: ${{ fromJson(needs.fetch_all_pull_request_shas.outputs.pr_details) }} with: @@ -112,7 +111,7 @@ jobs: strategy: fail-fast: false matrix: - # Existing LTI release tags can be added to obtain build results. + # Existing LTI release tags and branches can be added to obtain build results.If the tag array is empty, it will default to 'main'. However, if there is at least one tag or branch in the tag array and you need to run on 'main' as well, make sure to add 'main' to the array. tag: [''] with: useLocalPlugin: true From 011faa4176dd7c9705f342a9a6b6ccfeb8a9f66b Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Mon, 15 Jul 2024 12:23:34 +0530 Subject: [PATCH 14/22] Specified Correct branch and removed Parameter 'specifyRepo' --- .github/workflows/cronJob.yaml | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 332b413c7..4d78788cc 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -95,7 +95,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags and branches can be added to obtain build results.If the tag array is empty, it will default to 'main'. However, if there is at least one tag or branch in the tag array and you need to run on 'main' as well, make sure to add 'main' to the array. - tag: [ '' ] # Can specify tags or branches such as '24.0.6' and 'main' + tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' pr_details: ${{ fromJson(needs.fetch_all_pull_request_shas.outputs.pr_details) }} with: useLocalPlugin: true @@ -104,7 +104,6 @@ jobs: refLTITag: ${{ matrix.tag }} name: Run PR - # Run the LTI Tests against lsp4ij main branch call-build-workflow-for-lsp4ij-main-branch: uses: ./.github/workflows/build.yaml @@ -112,7 +111,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags and branches can be added to obtain build results.If the tag array is empty, it will default to 'main'. However, if there is at least one tag or branch in the tag array and you need to run on 'main' as well, make sure to add 'main' to the array. - tag: [''] + tag: [ 'lsp4ij-market-0.0.2-integration' ] with: useLocalPlugin: true refLsp4ij: main From 118f65e34226cf6bd917ecd80259b511ee8c77b5 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Thu, 18 Jul 2024 14:37:47 +0530 Subject: [PATCH 15/22] Update build.yaml --- .github/workflows/build.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 38ae3f426..cd47e626b 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -108,7 +108,7 @@ jobs: if: ${{ runner.os == 'Linux' && !failure() }} uses: actions/upload-artifact@v4.3.4 with: - name: liberty-tools-intellij-LTI-main-LSP4IJ-${{ env.LSP4IJ_BRANCH }} + name: liberty-tools-intellij-LTI-${{ env.REF_LTI_TAG }}-LSP4IJ-${{ env.LSP4IJ_BRANCH }} path: | ./**/*liberty-tools-intellij*.zip ./**/libs/*liberty-tools-intellij*.jar From d5e329547a48ebb89bb3613f46eadd88c86d6496 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Fri, 19 Jul 2024 14:30:12 +0530 Subject: [PATCH 16/22] given 'default' value if 'env.REF_LTI_TAG' is empty --- .github/workflows/build.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index cd47e626b..3c7459b53 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -108,7 +108,7 @@ jobs: if: ${{ runner.os == 'Linux' && !failure() }} uses: actions/upload-artifact@v4.3.4 with: - name: liberty-tools-intellij-LTI-${{ env.REF_LTI_TAG }}-LSP4IJ-${{ env.LSP4IJ_BRANCH }} + name: liberty-tools-intellij-LTI-${{ env.REF_LTI_TAG != '' && env.REF_LTI_TAG || 'default' }}-LSP4IJ-${{ env.LSP4IJ_BRANCH }} path: | ./**/*liberty-tools-intellij*.zip ./**/libs/*liberty-tools-intellij*.jar From a3a29dd48f5594044e810b1fbcc076bcc338bda7 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Tue, 23 Jul 2024 10:34:01 +0530 Subject: [PATCH 17/22] made change to the name of LSP4IJ_BRANCH env variable --- .github/workflows/build.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 3c7459b53..1a468c5a5 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -64,7 +64,7 @@ jobs: env: USE_LOCAL_PLUGIN: ${{ inputs.useLocalPlugin || false }} REF_LSP4IJ: ${{ inputs.refLsp4ij }} - LSP4IJ_BRANCH: ${{ inputs.lsp4ijBranch || 'latest' }} + LSP4IJ_BRANCH: ${{ inputs.lsp4ijBranch || 'default' }} REF_LTI_TAG: ${{ inputs.refLTITag }} steps: - name: Configure pagefile From 538ac3887a9970a6a4430cc409aee17bbd092444 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Thu, 8 Aug 2024 11:37:40 +0530 Subject: [PATCH 18/22] Simplified the usage of env.REF_LTI_TAG --- .github/workflows/build.yaml | 2 +- .github/workflows/cronJob.yaml | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 1a468c5a5..fa465df30 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -108,7 +108,7 @@ jobs: if: ${{ runner.os == 'Linux' && !failure() }} uses: actions/upload-artifact@v4.3.4 with: - name: liberty-tools-intellij-LTI-${{ env.REF_LTI_TAG != '' && env.REF_LTI_TAG || 'default' }}-LSP4IJ-${{ env.LSP4IJ_BRANCH }} + name: liberty-tools-intellij-LTI-${{ env.REF_LTI_TAG || 'default' }}-LSP4IJ-${{ env.LSP4IJ_BRANCH }} path: | ./**/*liberty-tools-intellij*.zip ./**/libs/*liberty-tools-intellij*.jar diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 4d78788cc..d5cd27c0a 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -1,6 +1,7 @@ name: Cron Job on: + push: workflow_dispatch: schedule: # The job runs at 15:45 UTC every Monday through Friday, which is 9:15 PM IST and 11:45 AM EST. @@ -139,5 +140,5 @@ jobs: curl -X POST -H 'Content-type: application/json' --data "{\"text\": \"Workflow ${{ github.workflow }} triggered by branch *${{ github.ref }}*. Build results: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}. Status: *$STATUS*\"}" $SLACK_WEBHOOK_URL env: - SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }} + SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL_COPY }} name: Run Slack Notification \ No newline at end of file From bb340b5eee6d6e1c598f379e17710a332e8c8c3b Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Thu, 8 Aug 2024 13:20:32 +0530 Subject: [PATCH 19/22] added code as part of testing --- .github/workflows/build.yaml | 2 +- .github/workflows/cronJob.yaml | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index fa465df30..5eb2c235b 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -121,6 +121,6 @@ jobs: if: ${{ failure() }} uses: actions/upload-artifact@v3 with: - name: ${{ matrix.reportName }} + name: ${{ matrix.reportName }}-LTI-${{ env.REF_LTI_TAG || 'default' }}-LSP4IJ-${{ env.LSP4IJ_BRANCH }} path: | liberty-tools-intellij/build/reports/ \ No newline at end of file diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index d5cd27c0a..d0f2e350f 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -96,7 +96,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags and branches can be added to obtain build results.If the tag array is empty, it will default to 'main'. However, if there is at least one tag or branch in the tag array and you need to run on 'main' as well, make sure to add 'main' to the array. - tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' + tag: [ 'lsp4ij-market-0.0.2-integration', 'main', 'v0.0.0.3', 'v0.0.0.1' ] # Can specify tags or branches such as '24.0.6' and 'main' pr_details: ${{ fromJson(needs.fetch_all_pull_request_shas.outputs.pr_details) }} with: useLocalPlugin: true @@ -112,7 +112,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags and branches can be added to obtain build results.If the tag array is empty, it will default to 'main'. However, if there is at least one tag or branch in the tag array and you need to run on 'main' as well, make sure to add 'main' to the array. - tag: [ 'lsp4ij-market-0.0.2-integration' ] + tag: [ 'lsp4ij-market-0.0.2-integration', , 'main', 'v0.0.0.3', 'v0.0.0.1' ] with: useLocalPlugin: true refLsp4ij: main From b98e38aabb23e876a2e0478d0c0fbdfc206436f8 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Thu, 8 Aug 2024 13:23:15 +0530 Subject: [PATCH 20/22] Update cronJob.yaml --- .github/workflows/cronJob.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index d0f2e350f..696923c7b 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -112,7 +112,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags and branches can be added to obtain build results.If the tag array is empty, it will default to 'main'. However, if there is at least one tag or branch in the tag array and you need to run on 'main' as well, make sure to add 'main' to the array. - tag: [ 'lsp4ij-market-0.0.2-integration', , 'main', 'v0.0.0.3', 'v0.0.0.1' ] + tag: [ 'lsp4ij-market-0.0.2-integration', 'main', 'v0.0.0.3', 'v0.0.0.1' ] with: useLocalPlugin: true refLsp4ij: main From dc138fabb4451eb5e16bbc9be64a3aab090e0bc1 Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Thu, 8 Aug 2024 16:51:18 +0530 Subject: [PATCH 21/22] Removed the code that was added for testing --- .github/workflows/build.yaml | 2 +- .github/workflows/cronJob.yaml | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml index 5eb2c235b..fa465df30 100644 --- a/.github/workflows/build.yaml +++ b/.github/workflows/build.yaml @@ -121,6 +121,6 @@ jobs: if: ${{ failure() }} uses: actions/upload-artifact@v3 with: - name: ${{ matrix.reportName }}-LTI-${{ env.REF_LTI_TAG || 'default' }}-LSP4IJ-${{ env.LSP4IJ_BRANCH }} + name: ${{ matrix.reportName }} path: | liberty-tools-intellij/build/reports/ \ No newline at end of file diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index 696923c7b..a3b4f0182 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -1,7 +1,6 @@ name: Cron Job on: - push: workflow_dispatch: schedule: # The job runs at 15:45 UTC every Monday through Friday, which is 9:15 PM IST and 11:45 AM EST. @@ -140,5 +139,5 @@ jobs: curl -X POST -H 'Content-type: application/json' --data "{\"text\": \"Workflow ${{ github.workflow }} triggered by branch *${{ github.ref }}*. Build results: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}. Status: *$STATUS*\"}" $SLACK_WEBHOOK_URL env: - SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL_COPY }} + SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }} name: Run Slack Notification \ No newline at end of file From 4caedf6401590702ddf0fe6d7f0a1b50f811b52c Mon Sep 17 00:00:00 2001 From: vaisakhkannan <133869023+vaisakhkannan@users.noreply.github.com> Date: Thu, 8 Aug 2024 16:53:34 +0530 Subject: [PATCH 22/22] cleaned up the code --- .github/workflows/cronJob.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/cronJob.yaml b/.github/workflows/cronJob.yaml index a3b4f0182..4d78788cc 100644 --- a/.github/workflows/cronJob.yaml +++ b/.github/workflows/cronJob.yaml @@ -95,7 +95,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags and branches can be added to obtain build results.If the tag array is empty, it will default to 'main'. However, if there is at least one tag or branch in the tag array and you need to run on 'main' as well, make sure to add 'main' to the array. - tag: [ 'lsp4ij-market-0.0.2-integration', 'main', 'v0.0.0.3', 'v0.0.0.1' ] # Can specify tags or branches such as '24.0.6' and 'main' + tag: [ 'lsp4ij-market-0.0.2-integration' ] # Can specify tags or branches such as '24.0.6' and 'main' pr_details: ${{ fromJson(needs.fetch_all_pull_request_shas.outputs.pr_details) }} with: useLocalPlugin: true @@ -111,7 +111,7 @@ jobs: fail-fast: false matrix: # Existing LTI release tags and branches can be added to obtain build results.If the tag array is empty, it will default to 'main'. However, if there is at least one tag or branch in the tag array and you need to run on 'main' as well, make sure to add 'main' to the array. - tag: [ 'lsp4ij-market-0.0.2-integration', 'main', 'v0.0.0.3', 'v0.0.0.1' ] + tag: [ 'lsp4ij-market-0.0.2-integration' ] with: useLocalPlugin: true refLsp4ij: main