Today's celebrity engagement....

They’ve been on again and off again for a couple years, and now it appears they’re on for good.

Modern Family star Sofia Vergara is reportedly engaged to her boyfriend of two years, Nick Loeb. The couple were holidaying in Mexico for her 40th birthday (with 90 loved ones) when he popped the question.

People magazine reports:

Nearly two months after news broke that Sofia Vergara and Nick Loeb were splitting up  he gave the Modern Family star a shiny new ring for her 40th birthday. One source tells PEOPLE the couple are engaged, but other sources say the actress does not consider herself engaged.

An unnamed Mexico-based rep for the star tells, “As of today, Sofia is not engaged.”

Loeb accompanied the actress to Mexico, where she celebrated her birthdaywith a weekend of festivities. On Monday, a Vergara pal posted a photo on Twitter of the actress wearing a sparkler on her ring finger in Mexico. Tuesday, Wonderwall reported the couple are engaged.

The pair met at the Golden Globes in 2010. They reportedly split in May of this year, but got back together a few weeks later. Sofia was previously married to Joe Gonzalez and had a 20-year-old son named Manolo.