You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2020/09/08 16:19:13 UTC
[GitHub] [incubator-nuttx-website] btashton opened a new pull request #31: CI: Add documentation on triggering the deployment workflow
btashton opened a new pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31
## Summary
Add documentation to the README on triggering the deployment workflow.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] v01d commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
v01d commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-697368094
@btashton we're still not getting response from INFRA on this, which is worrying. Also, is it your intention that they create a PAT for us and then add it as a secret or the PAT would be from one of us?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] v01d commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
v01d commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691524106
Cool, thanks. I see you also just pinged them.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] v01d commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
v01d commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691336061
> yeah eventually this trigger will get called from the OS CI, but we need secrets configured for that and INFRA was looking into it so it is a quick manual step for now.
>
> If the PR is against this repo then no additional action is needed.
Is there a link you can share to the request to INFRA? I would like to ping them to have this sorted out.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] v01d commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
v01d commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691522605
@btashton sorry to bother you, not sure if you were notified about my last comment
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691523572
@v01d sure https://issues.apache.org/jira/browse/INFRA-20805 it was originally just about the PAT for deploying our container images to GitHub Container Registry, but it has the same PAT secret issue. They said they needed to look into this, but I added a comment just not about this other use case explicitly.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-697481305
Yep you are looking for on.schedule
https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#onschedule
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691523572
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-688988398
@adamfeuer @v01d @Ouss4 since you all have been helping out with the documentation I figured I should point you at this and add it to the README.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] v01d commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
v01d commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691336061
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] adamfeuer commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
adamfeuer commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-689006862
Thanks @btashton – this is what we run to deploy the website, including documentation, is that right?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-697471395
@v01d do you want to give it a go? Otherwise I could pick it up over the weekend since I don't think there is a huge rush.
I'm thinking we just kick the build every two hour and don't worry about inspecting the commits in OS for changes since changes in some markdown external to the doc would also require a new build. It's not like these commits are that big and we could always prune the asf-site branch if needed.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] v01d commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
v01d commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-697474232
Sure, I can give it a try. I assume GH actions supports a cron job, right? I will open an issue for this.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton edited a comment on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton edited a comment on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-689007749
yeah eventually this trigger will get called from the OS CI, but we need secrets configured for that and INFRA was looking into it so it is a quick manual step for now.
If the PR is against this repo then no additional action is needed.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] v01d merged pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
v01d merged pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] v01d commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
v01d commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-697418531
That's a good idea
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] adamfeuer commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
adamfeuer commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-689008900
@btashton Cool, thank you. LGTM. 👍
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-697415928
> @btashton we're still not getting response from INFRA on this, which is worrying. Also, is it your intention that they create a PAT for us and then add it as a secret or the PAT would be from one of us?
My intention is they create one because otherwise the PAT will include our private repos due to scoping. I can more it back to waiting for infra from review so it goes to the top of the queue.
In the meantime we can also wire up a job that run every couple hours that syncs if there has been a change. Then it is automatic, and we still have the manual button if needed.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton edited a comment on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton edited a comment on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691523572
@v01d sure https://issues.apache.org/jira/browse/INFRA-20805 it was originally just about the PAT for deploying our container images to GitHub Container Registry, but it has the same PAT secret issue. They said they needed to look into this, but I added a comment just now about this other use case explicitly.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-689007749
yeah eventually this trigger will get called from the OS CI, but we need secrets configured for that and INFRA was looking into it so it is a quick manual step for now.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton edited a comment on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton edited a comment on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691523572
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton edited a comment on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton edited a comment on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691523572
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-nuttx-website] btashton commented on pull request #31: CI: Add documentation on triggering the deployment workflow
Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #31:
URL: https://github.com/apache/incubator-nuttx-website/pull/31#issuecomment-691523572
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org