-
Notifications
You must be signed in to change notification settings - Fork 86
CLOUDP-59015: Upload rpm to mongo yum repo #78
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
Conversation
.evergreen.yml
Outdated
@@ -122,6 +122,50 @@ tasks: | |||
permissions: public-read | |||
content_type: ${content_type|application/x-gzip} | |||
|
|||
- name: publish |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
call this push.
</html> | ||
|
||
|
||
repos: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd trim the list of repos to be only things you intend to actual push.
.evergreen.yml
Outdated
--arch x86_64 \ | ||
--packages https://s3.amazonaws.com/mongodb-mongocli-build/${project}/${revision}/dist/mongocli_next_${revision}_${version_id}_${created_at}_linux_x86_64.rpm | ||
|
||
./curator repo submit \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd do what you can to try and get org/community to run in parallel.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, I was just thinking of this seeing how long this takes, I may need to run other distros in parallel as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there's basically no local work, it's all busy-waiting, so finding some way to not serialize these tasks is good (and largely, all the work is parallelized on the backend.
@tychoish this is RFAL, opened a build ticket as it looks like we need some work for other rpm distros that are not rehad7 |
what is RFAL? |
.evergreen.yml
Outdated
run_on: | ||
- rhel70-small | ||
tasks: | ||
- name: compile | ||
- name: package | ||
# Redhat buildvariants |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd just append a bunch of tasks to a single build variant.
is the thing in the package only a go binary? does it link to any system library. If contents of the package are the same regardless, you should be able to push the same .rpm all RPM repositories |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see other comments
@tychoish changed this to be parallel tasks instead of buildvariants,
Or
And no clear idea or reference if this is just miss-configuration on our side |
@tychoish any other comment here? I also updated BUILD-10839 with more details |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Proposed changes
Jira ticket: CLOUDP-59015
Further comments
Running test here https://evergreen.mongodb.com/version/5e84863cd1fe0775bdea7566