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

ci(test): using docker images for test and updating to Slack orb v4 #333

Merged
merged 25 commits into from
Jul 29, 2021

Conversation

WikiRik
Copy link
Member

@WikiRik WikiRik commented Jul 19, 2021

I've finished the final things that I wanted to do for the CircleCI config and I've updated everything to the latest version.

I've also written a guide on what needs updating and where the latest versions can be found; https://myaegee.atlassian.net/l/c/CH731sTD

V4 of the Slack orb uses a different setup, so a new Slack app was created for this. The old one with webhooks can be removed after this PR is merged on all repos. A preview of the new Slack messages can be found here; https://github.com/CircleCI-Public/slack-orb/wiki

@WikiRik WikiRik requested a review from a team July 19, 2021 20:43
failure_message: The Docker image for \`$CIRCLE_PROJECT_REPONAME\` build and push has failed.
- slack/notify:
event: pass
template: basic_success_1
Copy link
Member

Choose a reason for hiding this comment

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

what will this output?

Copy link
Member Author

Choose a reason for hiding this comment

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

Something like this;

So basically it will tell us that docker-push-and-build passed for the master branch of discounts

Copy link
Member

Choose a reason for hiding this comment

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

can it be customized? it'd be nice for us to see which version was built and pushed, like in the setup I did

Copy link
Member Author

Choose a reason for hiding this comment

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

We can make custom notifications with the Block Kit Builder from Slack. I'll take a look to see if I can make something similar to what we have now

Copy link
Member Author

Choose a reason for hiding this comment

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

Okay, made a custom notification. Not sure how I could get the green bar in front that the old message had, but this should also suffice imo. Name and avatar of the app can be changed in Slack, no code needed for that
slack_2021-07-20_00-40-27

Copy link
Member Author

Choose a reason for hiding this comment

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

@serge1peshcoff is the message above good enough or do you want a visual indicator as well to see if it was successfull?

Copy link
Member

Choose a reason for hiding this comment

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

lgtm

@codecov
Copy link

codecov bot commented Jul 19, 2021

Codecov Report

Merging #333 (011f384) into master (9895bc0) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##            master      #333   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           16        16           
  Lines          286       286           
  Branches        36        36           
=========================================
  Hits           286       286           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9895bc0...011f384. Read the comment docs.

@WikiRik
Copy link
Member Author

WikiRik commented Jul 28, 2021

I was just talking to @linuxbandit about this PR and that I tried doing parallel tests before (few months back) but then ran into some issues. I tried it again and now it works! So we can test in parallel which should result in quicker finishing of CI checks (especially on core/statutory where tests take quite long)

@WikiRik WikiRik requested a review from a team July 28, 2021 19:02
@WikiRik WikiRik merged commit d1cb481 into master Jul 29, 2021
@WikiRik WikiRik deleted the circleci-test branch July 29, 2021 11:19
@serge1peshcoff
Copy link
Member

🎉 This PR is included in version 1.2.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

WikiRik added a commit to AEGEE/frontend that referenced this pull request Sep 27, 2021
Needed for semantic release, other updates like done in AEGEE/discounts#333 will be added later
WikiRik added a commit to AEGEE/frontend that referenced this pull request Sep 27, 2021
Needed for semantic release, other updates like done in AEGEE/discounts#333 will be added later
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants