Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Create runtime staging clone to manually kick off full test runs #61443

Merged
merged 14 commits into from
Nov 15, 2021

Conversation

steveisok
Copy link
Member

@steveisok steveisok commented Nov 11, 2021

This change allows devs to manually kick off full test runs on the configurations that only execute smoke tests per PR.

/azp run runtime-staging-manual will do the trick

@ghost
Copy link

ghost commented Nov 11, 2021

Tagging subscribers to this area: @Anipik, @safern, @ViktorHofer
See info in area-owners.md if you want to be subscribed.

Issue Details

null

Author: steveisok
Assignees: -
Labels:

area-Infrastructure-libraries

Milestone: -

@safern
Copy link
Member

safern commented Nov 11, 2021

/azp run manual-runtime-staging

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@steveisok
Copy link
Member Author

/azp run manual-runtime-staging

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Co-authored-by: Santiago Fernandez Madero <[email protected]>
@steveisok
Copy link
Member Author

/azp run manual-runtime-staging

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@steveisok steveisok force-pushed the stpfiste/manual-runtime-staging branch from c97977a to 3ebaa8c Compare November 11, 2021 02:57
@steveisok
Copy link
Member Author

/azp run manual-runtime-staging

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@steveisok
Copy link
Member Author

/azp run manual-runtime-staging

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@steveisok steveisok force-pushed the stpfiste/manual-runtime-staging branch 4 times, most recently from 71b05ee to 5267f76 Compare November 12, 2021 03:28
@steveisok steveisok force-pushed the stpfiste/manual-runtime-staging branch from 5267f76 to 44e7023 Compare November 12, 2021 03:32
@steveisok
Copy link
Member Author

@safern @akoeplinger conditioning the schedule based on pipeline name doesn't seem to work. Any suggestions? I think this might be where we actually need a separate yml.

- ${{ if not(eq(variables['Build.DefinitionName'], 'runtime-staging-manual')) }}:
  schedules:
    - cron: "0 7,19 * * *" # run at 7:00 and 19:00 (UTC) which is 23:00 and 11:00 (PST).
      displayName: Runtime-staging default schedule
      branches:
        include:
        - main
      always: false # run only if there were changes since the last successful scheduled run.

@akoeplinger akoeplinger force-pushed the stpfiste/manual-runtime-staging branch 3 times, most recently from f13eacb to 10b3299 Compare November 12, 2021 17:13
@akoeplinger akoeplinger force-pushed the stpfiste/manual-runtime-staging branch from 10b3299 to b497b31 Compare November 12, 2021 17:15
@akoeplinger
Copy link
Member

/azp run runtime-staging-manual

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

trigger: none

extends:
template: runtime-staging-template.yml
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

now that we have a template, maybe we can use parameters to turn on/off legs rather than checking based on pipeline name?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like platform!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that works for runtime-staging but we still have runtime.yml, so I'd need to move that to template as well which I'm not sure is desired.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That one also has a schedule trigger, so it will limit us as well and we will need the template anyway?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is a good first step. I think we can get more custom in a follow up PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants