driazati commented on code in PR #11594:
URL: https://github.com/apache/tvm/pull/11594#discussion_r893694711


##########
jenkins/Test.groovy.j2:
##########
@@ -266,8 +266,13 @@ stage('Test') {
             )
           }
           {{ m.upload_artifacts(tag='docs', filenames=["docs.tgz"]) }}
+          def docs_path = 
"tvm-jenkins-artifacts-prod/tvm/${env.BRANCH_NAME}/docs/"
           sh(
-            script: "aws s3 cp --no-progress _docs s3://${s3_prefix}/docs 
--recursive",
+            script: """
+              set -eux
+              aws s3 rm --no-progress s3://${docs_path} --recursive

Review Comment:
   the `@tvm-bot <command>` workflow used to do this: 
https://github.com/apache/tvm/blob/main/tests/scripts/github_tvmbot.py#L71, it 
queries the PR for the latest N comments then iterates over them looking at the 
contents. Here I think it would make sense to:
   1. query for the first few comments with graphql and something like
   ```graphql
   query ($owner: String!, $name: String!, $number: Int!) {
     repository(owner: $owner, name: $name) {
       pullRequest(number: $number) {
         comments(first: 100) {
           nodes {
             author {
               login
             }
             body
             databaseId
           }
         }
       }
     }
   }
   ```
   
   2. search for a sigil that the bot leaves in its comments, maybe an HTML 
comment like `<!---tvm-bot comment marker -->`
   3. update it in place via the `databaseId` if found, post a new comment if 
not



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to