Skip to content

Fix require for oncall scripts#9105

Merged
zachmargolis merged 1 commit intomainfrom
margolis-fix-require-for-cloudwatch-scripts
Aug 28, 2023
Merged

Fix require for oncall scripts#9105
zachmargolis merged 1 commit intomainfrom
margolis-fix-require-for-cloudwatch-scripts

Conversation

@zachmargolis
Copy link
Contributor

Somehow I broke scripts that use the cloudwatch client by adding this require in #9098

before:

identity-idp:main> aws-vault exec prod-power -- ./bin/oncall/email-deliveries c7abcdef
<internal:/.rbenv/versions/3.2.2/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:148:in `require': cannot load such file -- identity/hostdata (LoadError)

after:

> aws-vault exec prod-power -- ./bin/oncall/email-deliveries abcdef
[ Querying logs ] --=---=---=---=---=---=---=---=-----=---=---

@zachmargolis zachmargolis requested review from a team August 28, 2023 20:57
@zachmargolis zachmargolis merged commit 8b4ed2f into main Aug 28, 2023
@zachmargolis zachmargolis deleted the margolis-fix-require-for-cloudwatch-scripts branch August 28, 2023 21:27
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