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.

Sign up for GitHub

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

Jump to bottom

Replace Azure Pipelines with Gitlab CI #96

Merged
merged 16 commits into from
Oct 18, 2019
Merged

Replace Azure Pipelines with Gitlab CI #96

merged 16 commits into from
Oct 18, 2019

Conversation

esteve
Copy link
Contributor

@esteve esteve commented Oct 11, 2019 •
edited

Removed Azure Pipelines and added Gitlab CI in preparation for the Gitlab migration.

This change is Reviewable

@esteve esteve mentioned this pull request Oct 11, 2019
Open
6 tasks
@esteve esteve changed the title WIP Revert "Removed Gitlab CI" Replace Azure Pipelines with Gitlab CI Oct 14, 2019
@esteve
Copy link
Contributor Author

esteve commented Oct 14, 2019

@daggarwa I've readded Gitlab CI and also added a new job for cross compiling performance_test

@daggarwa
Copy link

@esteve Will this fail #90 if we merge in the Gitlab CI since its not currently containing any job to install Cyclone DDS whereas Azure did?

@daggarwa
Copy link

daggarwa commented Oct 15, 2019 •
edited

@esteve The CI pipeline is failing currently due to Cyclone DDS Mavin dependency. Can you please check what is the issue. Thanks

@daggarwa
Copy link

@esteve The CI pipeline is failing now due to $ cp LICENSE /opt/performance_test cp: cannot stat 'LICENSE': No such file or directory . I am not sure what is this error about.

@esteve
Copy link
Contributor Author

esteve commented Oct 17, 2019

@daggarwa Thanks. I'm working on it, but didn't have time to finish it yesterday.

@esteve
Copy link
Contributor Author

esteve commented Oct 17, 2019

@daggarwa I've fixed the last issue and the Gitlab CI now passes. I can't remove the Azure trigger until this PR has been merged, so it's fine the second check is still on hold.

@jpsamper2009
Copy link
Contributor

Since we can't clear the status above (which refers to the old azure pipeline), I'll merge this manually.

@jpsamper2009 jpsamper2009 merged commit 2e8e44e into master Oct 18, 2019
@esteve esteve deleted the revert-61-remove-gitlab-ci branch October 18, 2019 08:31
@esteve
Copy link
Contributor Author

esteve commented Oct 18, 2019

@jpsamper2009 thanks!

@daggarwa we'll have to update the other PRs to use the new CI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daggarwa daggarwa daggarwa approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@esteve @daggarwa @jpsamper2009