Skip to content

Generate missing or out of sync metadata.json#331

Closed
skywalk7 wants to merge 2 commits intofacebook:mainfrom
skywalk7:export-D75697334
Closed

Generate missing or out of sync metadata.json#331
skywalk7 wants to merge 2 commits intofacebook:mainfrom
skywalk7:export-D75697334

Conversation

@skywalk7
Copy link
Contributor

Summary:
X-link: facebook/IT-CPE#283

Automatically generated metadata.json using knife cookbook metadata --all

Differential Revision: D75697334

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D75697334

skywalk7 added a commit to skywalk7/IT-CPE that referenced this pull request Jul 24, 2025
Summary:
X-link: facebook/chef-cookbooks#331


Automatically generated metadata.json using `knife cookbook metadata --all`

Differential Revision: D75697334
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D75697334

skywalk7 added a commit to skywalk7/chef-cookbooks that referenced this pull request Jul 24, 2025
Summary:
Pull Request resolved: facebook#331

X-link: facebook/IT-CPE#283

Automatically generated metadata.json using `knife cookbook metadata --all`

Differential Revision: D75697334
skywalk7 added a commit to skywalk7/IT-CPE that referenced this pull request Jul 24, 2025
Summary:
X-link: facebook/chef-cookbooks#331

Pull Request resolved: facebook#283

Automatically generated metadata.json using `knife cookbook metadata --all`

Differential Revision: D75697334
Copy link
Collaborator

@jaymzh jaymzh left a comment

Choose a reason for hiding this comment

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

You said automatic... is there some process that updates these automatically when metadata.rb is updated? I'm fine with that being an internal thing, but I'm just curious. If not, should we have some check that there's no delta on them?

skywalk7 added a commit to skywalk7/IT-CPE that referenced this pull request Jul 25, 2025
Summary:
X-link: facebook/chef-cookbooks#331


Automatically generated metadata.json using `knife cookbook metadata --all`

Differential Revision: D75697334
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D75697334

skywalk7 added a commit to skywalk7/chef-cookbooks that referenced this pull request Jul 25, 2025
Summary:
Pull Request resolved: facebook#331

X-link: facebook/IT-CPE#283

Automatically generated metadata.json using `knife cookbook metadata --all`

Differential Revision: D75697334
skywalk7 added a commit to skywalk7/IT-CPE that referenced this pull request Jul 25, 2025
Summary:
X-link: facebook/chef-cookbooks#331

Pull Request resolved: facebook#283

Automatically generated metadata.json using `knife cookbook metadata --all`

Differential Revision: D75697334
skywalk7 added a commit to skywalk7/IT-CPE that referenced this pull request Jul 25, 2025
Summary:
X-link: facebook/chef-cookbooks#331


Automatically generated metadata.json using `knife cookbook metadata --all`

Differential Revision: D75697334
Summary:
Pull Request resolved: facebook#331

X-link: facebook/IT-CPE#283

Automatically generated metadata.json using `knife cookbook metadata --all`

Differential Revision: D75697334
skywalk7 added a commit to skywalk7/IT-CPE that referenced this pull request Jul 25, 2025
Summary:
X-link: facebook/chef-cookbooks#331

Pull Request resolved: facebook#283

Automatically generated metadata.json using `knife cookbook metadata --all`

Differential Revision: D75697334
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D75697334

@skywalk7
Copy link
Contributor Author

You said automatic... is there some process that updates these automatically when metadata.rb is updated? I'm fine with that being an internal thing, but I'm just curious. If not, should we have some check that there's no delta on them?

Yes, we do have autocorrect lint tool. It's just initial push that's semi-auto.

P.S. Sorry for the noise.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants