Skip to content

Comments

fix CI tests to use aws assumed role#87

Merged
Asad Manzoor (asadmg) merged 1 commit intomainfrom
fix/ci-workflow-aws-assumed-role-and-spark
Jul 23, 2025
Merged

fix CI tests to use aws assumed role#87
Asad Manzoor (asadmg) merged 1 commit intomainfrom
fix/ci-workflow-aws-assumed-role-and-spark

Conversation

@asadmg
Copy link
Contributor

Description

This PR:

  • Fixes the AWS connection (by using an assumed role)
  • Enables spark to work with the above AWS connection

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation Update
  • 🎨 Style
  • 🧑‍💻 Code Refactor
  • 🔥 Performance Improvements
  • ✅ Test
  • 🤖 Build
  • 🔁 CI
  • 📦 Chore (Release)
  • ⏩ Revert

Related Tickets & Documents

Checklist

  • 💣 Is your change a breaking change?
  • 📖 I have updated the CHANGELOG.md

Added tests?

  • 👍 yes
  • 🙅 no, because they aren't needed
  • 🙋 no, because I need help

Added to documentation?

  • 📓 internal package docs (ymls, macros, readme, if applicable)
  • 📕 I have raised a Snowplow documentation PR if applicable (Link here if required)
  • 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

Copy link

@gthomson31 Greg Thomson (gthomson31) left a comment

Choose a reason for hiding this comment

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

LGTM

@asadmg Asad Manzoor (asadmg) merged commit de6c6bf into main Jul 23, 2025
11 of 12 checks passed
@asadmg Asad Manzoor (asadmg) deleted the fix/ci-workflow-aws-assumed-role-and-spark branch July 23, 2025 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants